On Mon, May 8, 2017 at 3:49 PM, Richard Biener
<richard.guent...@gmail.com> wrote:
> On Mon, May 8, 2017 at 2:41 PM, Wilco Dijkstra <wilco.dijks...@arm.com> wrote:
>> This fixes a few failures on ARM and AArch64 due to a recent change in
>> alignment peeling by switching the vector cost model off
>> (https://gcc.gnu.org/ml/gcc-patches/2017-05/msg00407.html).
>>
>> Tested on AArch64, ARM and x64 - committed as obvious.
>
> Thanks. Note that I'm not sure what -fno-vect-cost-model actually means when
> peeling for alignment, so this fix might not "prevail".
This broke the testcases -- you have to use dg-additional-options,
otherwise I now get
gcc.dg/vect/vect-44.c -flto -ffat-lto-objects : dump file does not exist
UNRESOLVED: gcc.dg/vect/vect-44.c -flto -ffat-lto-objects
scan-tree-dump-times vect "Alignment of access forced using peeling" 1
...
fixing that now.
Richard.
> Richard.
>
>> ChangeLog:
>> 2017-05-08 Wilco Dijkstra <wdijk...@arm.com>
>>
>> * testsuite/gcc.dg/vect/vect-44.c: Add -fno-vect-cost-model.
>> * gcc/testsuite/gcc.dg/vect/vect-50.c: Likewise.
>> --
>>
>> diff --git a/gcc/testsuite/gcc.dg/vect/vect-44.c
>> b/gcc/testsuite/gcc.dg/vect/vect-44.c
>> index
>> 186f9cfc9e26d6eb53514dec0fac176d696ec578..fbc593572429422c8e527c5e5559c515efd38aa6
>> 100644
>> --- a/gcc/testsuite/gcc.dg/vect/vect-44.c
>> +++ b/gcc/testsuite/gcc.dg/vect/vect-44.c
>> @@ -1,4 +1,5 @@
>> /* { dg-require-effective-target vect_float } */
>> +/* { dg-options "-fno-vect-cost-model" } */
>>
>> #include <stdarg.h>
>> #include "tree-vect.h"
>> diff --git a/gcc/testsuite/gcc.dg/vect/vect-50.c
>> b/gcc/testsuite/gcc.dg/vect/vect-50.c
>> index
>> 78bfd8d3920445fe51c7393a82870ea85f62bb55..0d5febc165ee3bf3b7b595237168d9d4b9604d4b
>> 100644
>> --- a/gcc/testsuite/gcc.dg/vect/vect-50.c
>> +++ b/gcc/testsuite/gcc.dg/vect/vect-50.c
>> @@ -1,4 +1,5 @@
>> /* { dg-require-effective-target vect_float } */
>> +/* { dg-options "-fno-vect-cost-model" } */
>>
>> #include <stdarg.h>
>> #include "tree-vect.h"
