From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id 8C2AF385E02B; Tue, 24 May 2022 09:37:43 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 8C2AF385E02B From: "rguenther at suse dot de" To: gcc-bugs@gcc.gnu.org Subject: [Bug tree-optimization/105705] [12/13 Regression] std::equal triggers incorrect -Wnonnull warning Date: Tue, 24 May 2022 09:37:43 +0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: tree-optimization X-Bugzilla-Version: 12.1.0 X-Bugzilla-Keywords: diagnostic X-Bugzilla-Severity: normal X-Bugzilla-Who: rguenther at suse dot de X-Bugzilla-Status: NEW X-Bugzilla-Resolution: X-Bugzilla-Priority: P2 X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org X-Bugzilla-Target-Milestone: 12.2 X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-BeenThere: gcc-bugs@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-bugs mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 24 May 2022 09:37:43 -0000 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D105705 --- Comment #7 from rguenther at suse dot de --- On Tue, 24 May 2022, redi at gcc dot gnu.org wrote: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D105705 >=20 > --- Comment #5 from Jonathan Wakely --- > The warning started to be given without -Wsystem-headers with r12-1992 >=20 > It was already present with -Wsystem-headers, but suppressed by default. Skimming that it looks for whether the inline stack contains _only_ system headers now which means that any system header content inlined into user code will now be warned on without -Wsystem-header. That might be OK if the system header code is just abstraction but for more complicated code it's going to expose details not helpful to the user. We might want to change this to set m_allsyslocs to true if the "tail" of the inline stack is in system header which boils down to asking it for the original location - the intent wasn't to do extra suppression (like for user code inlined into system header context) but to expose more code to diagnostics which we are not ready to do [in late diagnostics at least].=