From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from simark.ca by simark.ca with LMTP id JKaTLE3D/2PCywEAWB0awg (envelope-from ) for ; Wed, 01 Mar 2023 16:27:41 -0500 Received: by simark.ca (Postfix, from userid 112) id A4B8B1E223; Wed, 1 Mar 2023 16:27:41 -0500 (EST) Authentication-Results: simark.ca; dkim=pass (1024-bit key; secure) header.d=sourceware.org header.i=@sourceware.org header.a=rsa-sha256 header.s=default header.b=E4zm7j90; dkim-atps=neutral X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on simark.ca X-Spam-Level: X-Spam-Status: No, score=-5.3 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,RCVD_IN_DNSWL_MED, URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 Received: from sourceware.org (server2.sourceware.org [8.43.85.97]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by simark.ca (Postfix) with ESMTPS id D4CE31E0D3 for ; Wed, 1 Mar 2023 16:27:40 -0500 (EST) Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 8F611385B529 for ; Wed, 1 Mar 2023 21:27:39 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 8F611385B529 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1677706059; bh=+yL+tZr3H8YwkZ65DGmGFLTDeli/X7lRfuhiwzz8Gsw=; h=To:Cc:Subject:Date:List-Id:List-Unsubscribe:List-Archive: List-Post:List-Help:List-Subscribe:From:Reply-To:From; b=E4zm7j90gdRv5eiVkjaB2aT1js76EP7v2kc7W/TPR5vifA99IWsv49pKEchsxR9hH EWS7bJ0Q8B65tXqVSsyQOt91vI8mZggVC+bM6nhJJocayh2dweS5qfj7oqFndjDGOx LYkjneTqrByi1zrP5gd9Ywp12yMEQPoq5dMRUvLs= Received: from mail-il1-x131.google.com (mail-il1-x131.google.com [IPv6:2607:f8b0:4864:20::131]) by sourceware.org (Postfix) with ESMTPS id A2D21385084F for ; Wed, 1 Mar 2023 21:27:19 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org A2D21385084F Received: by mail-il1-x131.google.com with SMTP id i12so9211730ila.5 for ; Wed, 01 Mar 2023 13:27:19 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1677706039; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=+yL+tZr3H8YwkZ65DGmGFLTDeli/X7lRfuhiwzz8Gsw=; b=Eb4/cqFM0uwNUA0YX6nOFiC3bBU1B44Nzg5/Nh8Km29Nz2bkYu2cqvtJjtktt+RhUq peIa1JpIYA0VwwD+rBUd3UPj0c3XRS6z3fORX2Pel1eA4XpLBqz6ucJITA8h+5tA90CE 2zPnuVqN87BctzqVe7FQQ37gfP6ODe59XelbqhA5UfEDFacClkFUETdytLQSgDVDt2nS mcMf+yN0ZgYzTJY0q1+2MHhAeiZ4tTfxkIxHSRGKgGqo595EuFW856l+uE5bt6zFcS47 /Wk88NOsGt3X/JZOO1EEknlEnna/YzlegNryy7HZdhSsYxVmKx1c9PNCibbUYsjAfUAa xVjA== X-Gm-Message-State: AO0yUKX+1HpzeTKHPXVziij+TIrDP8aOURcS5oCuIiRR/hYNyM5kdT5a k1RNGnKneBmsQAMNOas7RQ8bblkimcDUOW5S X-Google-Smtp-Source: AK7set8VUWHpJR1/Iy4T24epPqq2znYbhds8BPfMAoCRfU3MtgSSLcX9jkhyDZ7ZBjROkZtRdB4bqQ== X-Received: by 2002:a05:6e02:1b0d:b0:315:4f67:7055 with SMTP id i13-20020a056e021b0d00b003154f677055mr7539975ilv.1.1677706038857; Wed, 01 Mar 2023 13:27:18 -0800 (PST) Received: from localhost.localdomain (75-166-130-93.hlrn.qwest.net. [75.166.130.93]) by smtp.gmail.com with ESMTPSA id k9-20020a02ccc9000000b003c4f3850000sm4243798jaq.49.2023.03.01.13.27.17 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 01 Mar 2023 13:27:17 -0800 (PST) To: gdb-patches@sourceware.org Cc: Tom Tromey Subject: [PATCH] Remove dead code from scalar_binop Date: Wed, 1 Mar 2023 14:27:08 -0700 Message-Id: <20230301212708.4030388-1-tromey@adacore.com> X-Mailer: git-send-email 2.39.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Tom Tromey via Gdb-patches Reply-To: Tom Tromey Errors-To: gdb-patches-bounces+public-inbox=simark.ca@sourceware.org Sender: "Gdb-patches" 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. --- gdb/valarith.c | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/gdb/valarith.c b/gdb/valarith.c index 6ea0cd9e8d1..ad53febcedc 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.39.1