This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Merge Ignore and Deprecated in .opt files.
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Martin Liška <mliska at suse dot cz>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Thu, 19 Jul 2018 15:47:13 +0200
- Subject: Re: [PATCH] Merge Ignore and Deprecated in .opt files.
- References: <53c7c87f-3e04-f7e7-d048-973e9d2be4ba@suse.cz>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Thu, Jul 19, 2018 at 03:25:15PM +0200, Martin Liška wrote:
> Few weeks ago I added new Deprecated flag for options. Apparently, there's
> one similar called Ignore. Thus I moved all Deprecated to Ignore and for
> all Ignored I do a warning of following format:
>
> $ xgcc: warning: switch ‘-mmpx’ is no longer supported
Is that what we want for all the Ignore options? Looking at {,*/}*.opt,
I see a lot of options that have those Warn(switch %qs is no longer supported)
and a lot of them that do not, while with your patch it will now warn all.
Especially when it is a warning without corresponding -W... option that can
be quite nasty.
Wouldn't be better to just remove Deprecated and keep the Ignore behavior it
had? Or make Deprecated effectively an alias for
Ignore Warn(switch %qs is no longer supported)
and use it for those options that did that?
Jakub