Re: [PATCH] Fix PR c++/92024

2019-10-30 Thread Jason Merrill
On 10/12/19 2:10 PM, Bernd Edlinger wrote: On 10/11/19 6:31 PM, Jason Merrill wrote: On 10/10/19 2:06 PM, Bernd Edlinger wrote: On 10/10/19 7:49 PM, Jason Merrill wrote: On 10/10/19 10:42 AM, Bernd Edlinger wrote: Hi, this fixes a crash when -Wshadow=compatible-local is enabled in the testca

[PING**2] [PATCH] Fix PR c++/92024

2019-10-26 Thread Bernd Edlinger
Ping... On 10/18/19 3:19 PM, Bernd Edlinger wrote: > Ping... > > for the c++ FE and testsuite changes in the updated patch > here: https://gcc.gnu.org/ml/gcc-patches/2019-10/msg00916.html > > > Thanks > Bernd. > > > > > On 10/12/19 8:10 PM, Bernd Edlinger wrote: >> On 10/11/19 6:31 PM, Jaso

Re: [PATCH] Fix PR c++/92024

2019-10-18 Thread Bernd Edlinger
Ping... for the c++ FE and testsuite changes in the updated patch here: https://gcc.gnu.org/ml/gcc-patches/2019-10/msg00916.html Thanks Bernd. On 10/12/19 8:10 PM, Bernd Edlinger wrote: > On 10/11/19 6:31 PM, Jason Merrill wrote: >> On 10/10/19 2:06 PM, Bernd Edlinger wrote: >>> On 10/10/19

Re: [PATCH] Fix PR c++/92024

2019-10-12 Thread Sandra Loosemore
On 10/12/19 12:10 PM, Bernd Edlinger wrote: [snip snip] So instead I'd like to adjust the doc of -Wshadow to reflect the implementation and remove the if(warn_shadow_local) to have C and C++ behave identical and hopefully now in sync with the doc. Bootstrapped and reg-tested on x86_64-pc-linu

Re: [PATCH] Fix PR c++/92024

2019-10-12 Thread Bernd Edlinger
On 10/11/19 6:31 PM, Jason Merrill wrote: > On 10/10/19 2:06 PM, Bernd Edlinger wrote: >> On 10/10/19 7:49 PM, Jason Merrill wrote: >>> On 10/10/19 10:42 AM, Bernd Edlinger wrote: Hi, this fixes a crash when -Wshadow=compatible-local is enabled in the testcase g++.dg/parse/crash

Re: [PATCH] Fix PR c++/92024

2019-10-11 Thread Bernd Edlinger
On 10/11/19 6:31 PM, Jason Merrill wrote: > On 10/10/19 2:06 PM, Bernd Edlinger wrote: >> On 10/10/19 7:49 PM, Jason Merrill wrote: >>> On 10/10/19 10:42 AM, Bernd Edlinger wrote: Hi, this fixes a crash when -Wshadow=compatible-local is enabled in the testcase g++.dg/parse/crash

Re: [PATCH] Fix PR c++/92024

2019-10-11 Thread Jason Merrill
On 10/10/19 2:06 PM, Bernd Edlinger wrote: On 10/10/19 7:49 PM, Jason Merrill wrote: On 10/10/19 10:42 AM, Bernd Edlinger wrote: Hi, this fixes a crash when -Wshadow=compatible-local is enabled in the testcase g++.dg/parse/crash68.C Why does that flag cause this crash? gcc/cp/name-lookup.

Re: [PATCH] Fix PR c++/92024

2019-10-10 Thread Marek Polacek
On Thu, Oct 10, 2019 at 06:06:40PM +, Bernd Edlinger wrote: > On 10/10/19 7:49 PM, Jason Merrill wrote: > > On 10/10/19 10:42 AM, Bernd Edlinger wrote: > >> Hi, > >> > >> this fixes a crash when -Wshadow=compatible-local is > >> enabled in the testcase g++.dg/parse/crash68.C > > > > Why does t

Re: [PATCH] Fix PR c++/92024

2019-10-10 Thread Bernd Edlinger
On 10/10/19 7:49 PM, Jason Merrill wrote: > On 10/10/19 10:42 AM, Bernd Edlinger wrote: >> Hi, >> >> this fixes a crash when -Wshadow=compatible-local is >> enabled in the testcase g++.dg/parse/crash68.C > > Why does that flag cause this crash? > gcc/cp/name-lookup.c: if (warn_shadow)

Re: [PATCH] Fix PR c++/92024

2019-10-10 Thread Jason Merrill
On 10/10/19 10:42 AM, Bernd Edlinger wrote: Hi, this fixes a crash when -Wshadow=compatible-local is enabled in the testcase g++.dg/parse/crash68.C Why does that flag cause this crash? Jason

[PATCH] Fix PR c++/92024

2019-10-10 Thread Bernd Edlinger
Hi, this fixes a crash when -Wshadow=compatible-local is enabled in the testcase g++.dg/parse/crash68.C Bootstrapped and reg-tested on x86_64-pc-linux-gnu. Is it OK for trunk? Thanks Bernd. patch-pr92024.diff Description: patch-pr92024.diff