Re: [PATCH] Merge Ignore and Deprecated in .opt files.

2018-09-04 Thread Segher Boessenkool
On Tue, Aug 21, 2018 at 03:14:29PM +0200, Martin Liška wrote: > On 08/18/2018 12:24 AM, Segher Boessenkool wrote: > > Removing Init is *wrong* as far as I see; it changes things, anyway. > > Could you not have done this as a separate patch? > > It's already in, but it should be fine. Note that I a

Re: [PATCH] Merge Ignore and Deprecated in .opt files.

2018-08-21 Thread Martin Liška
t;> > >> >From 0a7d5cd6cd6ca0586a350b95cd8f6ded095ba9c8 Mon Sep 17 00:00:00 2001 >> From: marxin >> Date: Wed, 18 Jul 2018 13:40:24 +0200 >> Subject: [PATCH] Merge Ignore and Deprecated in .opt files. >> >> gcc/ChangeLog: >> >> 2018-07-18 Mar

Re: [PATCH] Merge Ignore and Deprecated in .opt files.

2018-08-17 Thread Segher Boessenkool
t; > did something useful? > > > > That's not intended, I fixed that. It the patch acceptable in form in which > it is? > > Thanks, > Martin > > >From 0a7d5cd6cd6ca0586a350b95cd8f6ded095ba9c8 Mon Sep 17 00:00:00 2001 > From: marxin > Date: Wed, 1

Re: [PATCH] Merge Ignore and Deprecated in .opt files.

2018-08-16 Thread Joseph Myers
On Thu, 16 Aug 2018, Martin Liška wrote: > On 08/15/2018 06:38 PM, Joseph Myers wrote: > > On Wed, 15 Aug 2018, Martin Liška wrote: > > > >> Ok, so you have very similar opinion as Jakub. Thus I'm sending new > >> version that preserves status quo, it only does: > > > > This is removing RejectN

Re: [PATCH] Merge Ignore and Deprecated in .opt files.

2018-08-16 Thread Martin Liška
ble in form in which it is? Thanks, Martin >From 0a7d5cd6cd6ca0586a350b95cd8f6ded095ba9c8 Mon Sep 17 00:00:00 2001 From: marxin Date: Wed, 18 Jul 2018 13:40:24 +0200 Subject: [PATCH] Merge Ignore and Deprecated in .opt files. gcc/ChangeLog: 2018-07-18 Martin Liska * common.opt: Remove Warn, Init and

Re: [PATCH] Merge Ignore and Deprecated in .opt files.

2018-08-15 Thread Joseph Myers
On Wed, 15 Aug 2018, Martin Liška wrote: > Ok, so you have very similar opinion as Jakub. Thus I'm sending new > version that preserves status quo, it only does: This is removing RejectNegative from some Deprecated options. Won't that result in the -fno-* variants of those options starting to

Re: [PATCH] Merge Ignore and Deprecated in .opt files.

2018-08-15 Thread Martin Liška
gs from test-suite, if a test was testing only that, then I removed the test 4) I simplifies warning reporting mechanism for Deprecated options. Patch can bootstrap on x86_64-linux-gnu and survives regression tests. Ready to be installed? Martin >From 68edcd24e0a12d21b6eaca68596a9e4d7fd8ae

Re: [PATCH] Merge Ignore and Deprecated in .opt files.

2018-07-26 Thread Joseph Myers
On Fri, 20 Jul 2018, Martin Liška wrote: > +C++ ObjC++ Ignore) Stray ')' at the end of this line. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH] Merge Ignore and Deprecated in .opt files.

2018-07-26 Thread Joseph Myers
On Thu, 19 Jul 2018, Martin Liška wrote: > I must admit that was my intention :) In my eyes it makes it more consistent > and > it gives consumers feedback about usage of an option that does nothing. > For x86_64 there's list of options that are Ignore and don't produce a > warning: The design

Re: [PATCH] Merge Ignore and Deprecated in .opt files.

2018-07-20 Thread Martin Liška
tached version of the patch. Patch can bootstrap on x86_64-linux-gnu and survives regression tests. Martin > > Jakub > >From 2698b0af3fbd7c0f0372513dc00673932c48b6cb Mon Sep 17 00:00:00 2001 From: marxin Date: Wed, 18 Jul 2018 13:40:24 +0200 Subject: [PATCH] Merge Ignore and Depreca

Re: [PATCH] Merge Ignore and Deprecated in .opt files.

2018-07-19 Thread Jakub Jelinek
On Thu, Jul 19, 2018 at 04:16:10PM +0200, Martin Liška wrote: > I must admit that was my intention :) In my eyes it makes it more consistent > and > it gives consumers feedback about usage of an option that does nothing. > For x86_64 there's list of options that are Ignore and don't produce a > w

Re: [PATCH] Merge Ignore and Deprecated in .opt files.

2018-07-19 Thread Martin Liška
On 07/19/2018 03:47 PM, Jakub Jelinek wrote: > 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 fol

Re: [PATCH] Merge Ignore and Deprecated in .opt files.

2018-07-19 Thread Jakub Jelinek
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

[PATCH] Merge Ignore and Deprecated in .opt files.

2018-07-19 Thread Martin Liška
Hi. 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 After that there were quite some usages f