From 349a125d16bec4a894e01494df10a2825195178e Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Wed, 1 Mar 2023 14:02:40 -0700 Subject: [PATCH] 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 --- 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; -- 2.30.2