Remove dead code from scalar_binop

scalar_binop has code for "&&" and "||", but I think this code can't
currently be run -- and, furthermore, it doesn't make sense to have
this code here, as the point of these operators is to short-circuit
evaluation.

This patch removes the dead code.

Regression tested on x86-64 Fedora 36.

Approved-by: Kevin Buettner <kevinb@redhat.com>

diff --git a/gdb/valarith.c b/gdb/valarith.c
index 4c2b44360bb1fc08521afd3e4a67d3a7d4a44061..e0a3461aaa2edcf36cd42d1a1febcaac2a62d5d0 100644 (file)
--- a/gdb/valarith.c
+++ b/gdb/valarith.c
@@ -1331,14 +1331,6 @@ scalar_binop (struct value *arg1, struct value *arg2, enum exp_opcode op)
              v = v1 ^ v2;
              break;
 
-           case BINOP_LOGICAL_AND:
-             v = v1 && v2;
-             break;
-
-           case BINOP_LOGICAL_OR:
-             v = v1 || v2;
-             break;
-
            case BINOP_MIN:
              v = v1 < v2 ? v1 : v2;
              break;
@@ -1492,14 +1484,6 @@ scalar_binop (struct value *arg1, struct value *arg2, enum exp_opcode op)
              v = v1 ^ v2;
              break;
 
-           case BINOP_LOGICAL_AND:
-             v = v1 && v2;
-             break;
-
-           case BINOP_LOGICAL_OR:
-             v = v1 || v2;
-             break;
-
            case BINOP_MIN:
              v = v1 < v2 ? v1 : v2;
              break;