Re: [PATCH] Do not error for no_sanitize attributes (PR sanitizer/82490).

2017-10-11 Thread Jakub Jelinek
On Wed, Oct 11, 2017 at 09:45:27AM +0200, Markus Trippelsdorf wrote: > On 2017.10.11 at 09:39 +0200, Jakub Jelinek wrote: > > On Wed, Oct 11, 2017 at 08:24:28AM +0200, Martin Liška wrote: > > > Hi. > > > > > > This changes error to a warning: > > > warning: ‘foobar’ attribute directive ignored [-W

Re: [PATCH] Do not error for no_sanitize attributes (PR sanitizer/82490).

2017-10-11 Thread Markus Trippelsdorf
On 2017.10.11 at 09:39 +0200, Jakub Jelinek wrote: > On Wed, Oct 11, 2017 at 08:24:28AM +0200, Martin Liška wrote: > > Hi. > > > > This changes error to a warning: > > warning: ‘foobar’ attribute directive ignored [-Wattributes] > > > > Patch can bootstrap on ppc64le-redhat-linux and survives reg

Re: [PATCH] Do not error for no_sanitize attributes (PR sanitizer/82490).

2017-10-11 Thread Jakub Jelinek
On Wed, Oct 11, 2017 at 08:24:28AM +0200, Martin Liška wrote: > Hi. > > This changes error to a warning: > warning: ‘foobar’ attribute directive ignored [-Wattributes] > > Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. What is the rationale for not warning? LLVM comp

[PATCH] Do not error for no_sanitize attributes (PR sanitizer/82490).

2017-10-10 Thread Martin Liška
Hi. This changes error to a warning: warning: ‘foobar’ attribute directive ignored [-Wattributes] Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. Ready to be installed? Martin gcc/ChangeLog: 2017-10-10 Martin Liska PR sanitizer/82490 * opts.c (par