Re: [PATCH] Do not allow -fgnu-tm w/ -fsanitize={kernel-,}address (PR sanitizer/81302).

2017-07-17 Thread Jakub Jelinek 
On Mon, Jul 17, 2017 at 11:18:02AM +0200, Richard Biener wrote:
> On Mon, Jul 10, 2017 at 11:35 AM, Martin Liška  wrote:
> > Hello
> >
> > Following patch disables -fgnu-tm with AddressSanitizer.
> > Patch can bootstrap on ppc64le-redhat-linux and survives regression tests.
> >
> > Ready to be installed?
> 
> I think it should be a sorry () but no strong opinion.

Certainly better sorry than error, but ideally it should be just supported.
Of course with nobody actively working on -fgnu-tm that might take a while.

Jakub

Re: [PATCH] Do not allow -fgnu-tm w/ -fsanitize={kernel-,}address (PR sanitizer/81302).

2017-07-17 Thread Richard Biener 
On Mon, Jul 10, 2017 at 11:35 AM, Martin Liška  wrote:
> Hello
>
> Following patch disables -fgnu-tm with AddressSanitizer.
> Patch can bootstrap on ppc64le-redhat-linux and survives regression tests.
>
> Ready to be installed?

I think it should be a sorry () but no strong opinion.

Thanks,
Richard.

> Martin
>
> gcc/ChangeLog:
>
> 2017-07-04  Martin Liska  
>
> PR sanitizer/81302
> * opts.c (finish_options): Do not allow -fgnu-tm
> w/ -fsanitize={kernel-,}address.
> ---
>  gcc/opts.c | 8 
>  1 file changed, 8 insertions(+)
>
>