[binutils-gdb] Remove dead code from scalar_binop

[binutils-gdb] Remove dead code from scalar_binop

[Tom Tromey]

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=349a125d16bec4a894e01494df10a2825195178e

commit 349a125d16bec4a894e01494df10a2825195178e
Author: Tom Tromey <tromey@adacore.com>
Date:   Wed Mar 1 14:02:40 2023 -0700

    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:
---
 gdb/valarith.c | 16 ----------------
 1 file changed, 16 deletions(-)

diff --git a/gdb/valarith.c b/gdb/valarith.c
index 4c2b44360bb..e0a3461aaa2 100644
--- 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;