From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id C6F0C3858D28; Thu, 26 Jan 2023 10:19:37 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org C6F0C3858D28 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1674728377; bh=PexDjIRDpp5/ovx/gEsk5JPkgc31VLokox0YxsS4PwY=; h=From:To:Subject:Date:In-Reply-To:References:From; b=v9Q+mUPEo8cif7k6q51QwB5u+nywymyCxKl1JuWV52vpmbTDZASya6tTcMiszmg+b WwmP4hMdABlqBvIzFxL9h4t5+5LRP+Mbv9CKPIl5oLmN1eL+qq77USk7M4G2PhvnYW xqzigZGUMNXPSRoueZk53W33ZbgwqjwNzfSGtbXU= From: "iains at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug modula2/108555] gm2_langhook_option_lang_mask causes all (unappropriate) C language options to be recognized Date: Thu, 26 Jan 2023 10:19:37 +0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: modula2 X-Bugzilla-Version: 13.0 X-Bugzilla-Keywords: X-Bugzilla-Severity: normal X-Bugzilla-Who: iains at gcc dot gnu.org X-Bugzilla-Status: UNCONFIRMED X-Bugzilla-Resolution: X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: gaius at gcc dot gnu.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: cc 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 List-Id: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D108555 Iain Sandoe changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |iains at gcc dot gnu.org --- Comment #2 from Iain Sandoe --- We do claim the ones that modula-2 actually uses in gm2-lang.opt (previous to r13-5373-g80cf2c5e8f496b we had a large and growing set to accommodate everything that the preprocessor needs). Because of that growing number of added options, this change was made intentionally... the idea was to claim and act on the C and Driver ones that are used to construct C-preprocessor command lines as an action separate fr= om handling the Modula-2 options. IIUC, the mechanism of adding them to the language opt is a convention rath= er than a correctness issue? Or is it that we should reject rather than ignoring the options that we do = not pass to the preprocessor? (I can revert to adding everything to the gm2-lang.opt .. it just seemed a = bit hacky)=