https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85487
rsandifo at gcc dot gnu.org <rsandifo at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |rsandifo at gcc dot gnu.org
--- Comment #12 from rsandifo at gcc dot gnu.org <rsandifo at gcc dot gnu.org>
---
(In reply to Jonathan Wakely from comment #9)
> (In reply to Jakub Jelinek from comment #7)
> > I would say that is a terrible design...
>
> Yes, I completely agree, but I don't see why GCC should be in the business
> of diagnosing other people's junk :-)
>
> Maybe Visual Studio's editor and VScode do have checks, just not the VC++
> compiler. And if so, then that's even more reason that we don't need GCC to
> do its own checking.
Agreed. And if people with strict linting requirements want
a warning about pragmas that are recognised but have no effect
on the compiler, we could still provide an option to do that
(but it shouldn't be in -Wall or even -Wextra).
That shouldn't be a requirement for this PR though, unless anyone
can show that someone somewhere really does want these pragmas to
generate a warning.
Jeff said at the end of the thread that he wouldn't mind
if someone else approves it, so it's probably worth posting
again. The patch LGTM FWIW: only (very) minor comment is that
the unused argument name in handle_pragma_region can be dropped.
I think the patch would need to wait for GCC 13 now though.
