From: Tom Tromey Date: Wed, 1 Mar 2023 21:02:40 +0000 (-0700) Subject: Remove dead code from scalar_binop X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=349a125d16bec4a894e01494df10a2825195178e;p=binutils-gdb.git 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 --- 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;