[Patch] Gate vect-mask-store-move-1.c correctly, and actually output the dump

[Richard Biener]

On Tue, Feb 9, 2016 at 12:24 PM, James Greenhalgh
<james.greenhalgh@arm.com> wrote:
>
> On Mon, Feb 08, 2016 at 03:24:14PM +0100, Richard Biener wrote:
>> On Mon, Feb 8, 2016 at 2:40 PM, James Greenhalgh
>> <james.greenhalgh@arm.com> wrote:
>> > On Mon, Feb 08, 2016 at 04:29:31PM +0300, Yuri Rumyantsev wrote:
>> >> Hi James,
>> >>
>> >> Thanks for reporting this issue.
>> >> I prepared slightly different patch since we don't need to add
>> >> tree-vect dump option - it is on by default for all tests in /vect
>> >> directory.
>> >
>> > Hm, I added that line as my test runs were showing:
>> >
>> >   UNRESOLVED: gcc.dg/vect/vect-mask-store-move-1.c: dump file does not exist
>> >
>> > I would guess the explicit
>> >
>> >   /* { dg-options "-O3" } */
>> >
>> > is clobbering the vect.exp setup of flags?
>>
>> Yes.  Use { dg-additional-options "-O3" } instead.
>
> I don't see why this test needs anything more than the default vect
> options anyway... In which case, the patch would look like this.
>
> Tested on x86-64 where the test passes, and on AArch64 where it is
> correctly skipped.
>
> OK?

Ok.

Richard.

> Thanks,
> James
>
> ---
> 2016-02-09  James Greenhalgh  <james.greenhalgh@arm.com>
>
>         * gcc.dg/vect/vect-mask-store-move-1.c: Drop dg-options directive,
>         gate check on x86_64/i?86.
>