On Tue, Feb 9, 2016 at 12:24 PM, James Greenhalgh
<james.greenha...@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.greenha...@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.greenha...@arm.com>
>
> * gcc.dg/vect/vect-mask-store-move-1.c: Drop dg-options directive,
> gate check on x86_64/i?86.
>
