Re: [PATCH] driver: Improve the generated help text for alias options with arguments

2020-03-20 Thread Lewis Hyatt via Gcc-patches
On Fri, Mar 20, 2020 at 01:16:42PM -0600, Martin Sebor wrote: > On 3/20/20 11:46 AM, Richard Sandiford wrote: > > Martin Sebor writes: > > > On 3/17/20 5:52 AM, Richard Sandiford wrote: > > > > Lewis Hyatt writes: > > > > > On Mon, Mar 16, 2020 at 06:11:08PM +, Richard Sandiford wrote: > > >

Re: [PATCH] driver: Improve the generated help text for alias options with arguments

2020-03-20 Thread Martin Sebor via Gcc-patches
On 3/20/20 11:46 AM, Richard Sandiford wrote: Martin Sebor writes: On 3/17/20 5:52 AM, Richard Sandiford wrote: Lewis Hyatt writes: On Mon, Mar 16, 2020 at 06:11:08PM +, Richard Sandiford wrote: Lewis Hyatt via Gcc-patches writes: ... FWIW there are three other options currently affe

Re: [PATCH] driver: Improve the generated help text for alias options with arguments

2020-03-20 Thread Richard Sandiford
Martin Sebor writes: > On 3/17/20 5:52 AM, Richard Sandiford wrote: >> Lewis Hyatt writes: >>> On Mon, Mar 16, 2020 at 06:11:08PM +, Richard Sandiford wrote: Lewis Hyatt via Gcc-patches writes: >>> ... > FWIW there are three other options currently affected by this change > (-Wi

Re: [PATCH] driver: Improve the generated help text for alias options with arguments

2020-03-20 Thread Martin Sebor via Gcc-patches
On 3/17/20 5:52 AM, Richard Sandiford wrote: Lewis Hyatt writes: On Mon, Mar 16, 2020 at 06:11:08PM +, Richard Sandiford wrote: Lewis Hyatt via Gcc-patches writes: ... FWIW there are three other options currently affected by this change (-Wimplicit-fallthrough, -fcf-protection, and -fli

Re: [PATCH] driver: Improve the generated help text for alias options with arguments

2020-03-20 Thread Richard Sandiford
Lewis Hyatt writes: > On Tue, Mar 17, 2020 at 11:52:13AM +, Richard Sandiford wrote: >> Lewis Hyatt writes: >> > On Mon, Mar 16, 2020 at 06:11:08PM +, Richard Sandiford wrote: >> >> Lewis Hyatt via Gcc-patches writes: >> > ... >> >> > FWIW there are three other options currently affected

Re: [PATCH] driver: Improve the generated help text for alias options with arguments

2020-03-20 Thread Lewis Hyatt via Gcc-patches
On Tue, Mar 17, 2020 at 11:52:13AM +, Richard Sandiford wrote: > Lewis Hyatt writes: > > On Mon, Mar 16, 2020 at 06:11:08PM +, Richard Sandiford wrote: > >> Lewis Hyatt via Gcc-patches writes: > > ... > >> > FWIW there are three other options currently affected by this change > >> > (-Wim

Re: [PATCH] driver: Improve the generated help text for alias options with arguments

2020-03-17 Thread Richard Sandiford
Lewis Hyatt writes: > On Mon, Mar 16, 2020 at 06:11:08PM +, Richard Sandiford wrote: >> Lewis Hyatt via Gcc-patches writes: > ... >> > FWIW there are three other options currently affected by this change >> > (-Wimplicit-fallthrough, -fcf-protection, and -flive-patching). The change >> > for

Re: [PATCH] driver: Improve the generated help text for alias options with arguments

2020-03-16 Thread Lewis Hyatt via Gcc-patches
On Mon, Mar 16, 2020 at 6:11 PM Lewis Hyatt wrote: > Regarding -Wmissing-format-attribute, that is an interesting case, it's the > only > instance in any *.opt that has an = sign in the Alias target. If I understand > correctly, this one can't use the 3-argument form of Alias() because > -Wno-mis

Re: [PATCH] driver: Improve the generated help text for alias options with arguments

2020-03-16 Thread Lewis Hyatt via Gcc-patches
On Mon, Mar 16, 2020 at 06:11:08PM +, Richard Sandiford wrote: > Lewis Hyatt via Gcc-patches writes: ... > > FWIW there are three other options currently affected by this change > > (-Wimplicit-fallthrough, -fcf-protection, and -flive-patching). The change > > for > > -Wimplicit-fallthrough I

Re: [PATCH] driver: Improve the generated help text for alias options with arguments

2020-03-16 Thread Richard Sandiford
Lewis Hyatt via Gcc-patches writes: > Hello- > > Currently, if an option is both undocumented and an alias for a second option, > the help text generated by gcc --help directs the reader to use the other > option instead. This makes sense for deprecated options, but it seems this > pattern is also