From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id 295543858D32; Tue, 26 Dec 2023 18:24:30 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 295543858D32 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1703615070; bh=FFLlN3YSpzgoDdn/qPHHyLNH/bt9pDX1F4WQALwKZ/c=; h=From:To:Subject:Date:In-Reply-To:References:From; b=ch61RnMX6YcwUH5y9Ktwbjg2RmtETEEz8vOqNg4RMqUzXCHGorWgvpFcdWPc+vrUs DMGQkflCjEVJaUL99wiEKbB4TRotv3s4usV+Z9Yu96HnxTMrtKLrbeGoEdrKOG5vJ9 AERz4rl8DzWIjLG4mLmtSEIGwCBIu5ijZLFJy7n8= From: "ssbssa at sourceware dot org" To: gdb-prs@sourceware.org Subject: [Bug c++/17132] Improper evaluation of expressions involving virtual method invocations under EVAL_AVOID_SIDE_EFFECTS Date: Tue, 26 Dec 2023 18:24:28 +0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gdb X-Bugzilla-Component: c++ X-Bugzilla-Version: HEAD X-Bugzilla-Keywords: X-Bugzilla-Severity: normal X-Bugzilla-Who: ssbssa at sourceware dot org X-Bugzilla-Status: RESOLVED X-Bugzilla-Resolution: FIXED X-Bugzilla-Priority: P2 X-Bugzilla-Assigned-To: sivachandra at google dot com X-Bugzilla-Target-Milestone: 7.9 X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: cc resolution bug_status target_milestone Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://sourceware.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 List-Id: https://sourceware.org/bugzilla/show_bug.cgi?id=3D17132 Hannes Domani changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |ssbssa at sourceware dot o= rg Resolution|--- |FIXED Status|NEW |RESOLVED Target Milestone|--- |7.9 --- Comment #2 from Hannes Domani --- (In reply to Sourceware Commits from comment #1) > This is an automated email from the git hooks/post-receive script. It was > generated because a ref change was pushed to the repository containing > the project "gdb and binutils". >=20 > The branch, master has been updated > via e66d44466912ecf581f6b67ff299d064c7bf4f1a (commit) > from 940df408121be31beed22ef7a5ad133cb1592726 (commit) >=20 > Those revisions listed above that are new to this repository have > not appeared on any other notification email; so we list those > revisions in full, below. >=20 > - Log ----------------------------------------------------------------- > https://sourceware.org/git/gitweb.cgi?p=3Dbinutils-gdb.git; > h=3De66d44466912ecf581f6b67ff299d064c7bf4f1a >=20 > commit e66d44466912ecf581f6b67ff299d064c7bf4f1a > Author: Siva Chandra > Date: Wed Jul 9 10:25:48 2014 -0700 >=20 > Add new argument NOSIDE to find_overload_match. >=20=20=20=20=20 > This is a fix for PR c++/17132. > If this new argument is set to EVAL_AVOID_SIDE_EFFECTS, then the obje= ct's > memory will not be read while picking the best overload match. >=20=20=20=20=20 > gdb/ >=20=20=20=20=20 > * eval.c: Update all calls to find_overload_match. > * valarith.c: Likewise. > (value_user_defined_cpp_op, value_user_defined_op): New > argument NOSIDE. Update all callers. > * valops.c (find_overload_match): New argument NOSIDE. > * value.h (find_overload_match): Update signature. >=20=20=20=20=20 > gdb/testsuite >=20=20=20=20=20 > * gdb.cp/pr17132.cc: New file. > * gdb.cp/pr17132.exp: New file. >=20 > ----------------------------------------------------------------------- >=20 > Summary of changes: > gdb/ChangeLog | 10 ++++++ > gdb/eval.c | 6 ++-- > gdb/testsuite/ChangeLog | 6 ++++ > gdb/testsuite/gdb.cp/pr17132.cc | 61 > ++++++++++++++++++++++++++++++++++++++ > gdb/testsuite/gdb.cp/pr17132.exp | 40 +++++++++++++++++++++++++ > gdb/valarith.c | 15 +++++---- > gdb/valops.c | 19 +++++++++-- > gdb/value.h | 3 +- > 8 files changed, 146 insertions(+), 14 deletions(-) > create mode 100644 gdb/testsuite/gdb.cp/pr17132.cc > create mode 100644 gdb/testsuite/gdb.cp/pr17132.exp Was fixed by this commit. --=20 You are receiving this mail because: You are on the CC list for the bug.=