On Tue, Nov 13, 2018 at 04:01:56PM +0100, Martin Liška wrote:
> On 11/12/18 1:07 PM, Jakub Jelinek wrote:
> > On Mon, Nov 12, 2018 at 01:03:41PM +0100, Martin Liška wrote:
> >> The patch reject usage of the mentioned options.
> >>
> >> Ready for trunk?
> >> Thanks,
> >> Martin
> >>
> >> gcc/ChangeLog:
> >>
> >> 2018-11-12 Martin Liska <mli...@suse.cz>
> >>
> >> PR sanitizer/87930
> >> * config/i386/i386.c (ix86_option_override_internal): Error
> >> about usage -mabi=ms and -fsanitize={,kernel-}address.
> >
> > Please add testcases for this.
>
> Done in attached patch.
>
> Can this be changed through attribute too?
>
> No.
>
> I'm going to install the patch.
Can you please move the test to gcc.dg/asan/ and guard with
{ i?86-*-* x86_64-*-* } && lp64 ?
I don't think we don't want -fsanitize=address tests outside of /asan/
subdirs where we guard them on the availability of asan.
Jakub
