Hi all, This test started failing because some of the functions in the combine dump that it scans uses a different pattern to match the same instruction: insv_regsi rather than aarch64_bfi. The code generation is still the same. The patch changes the scan to look for the actual instruction we want in the assembly.
This fixes the test. Committing to trunk as obvious. Thanks, Kyrill 2018-09-14 Kyrylo Tkachov <kyrylo.tkac...@arm.com> * gcc.target/aarch64/combine_bfi_1.c: Scan for bfi instruction rather than pattern name in combine dump.
diff --git a/gcc/testsuite/gcc.target/aarch64/combine_bfi_1.c b/gcc/testsuite/gcc.target/aarch64/combine_bfi_1.c index 9cc3bdb3ddfc286bfb6c079aaba27fb5edb1806c..cee42ccafcce0e0524f30fa6857abd902ca5779a 100644 --- a/gcc/testsuite/gcc.target/aarch64/combine_bfi_1.c +++ b/gcc/testsuite/gcc.target/aarch64/combine_bfi_1.c @@ -1,5 +1,5 @@ /* { dg-do compile } */ -/* { dg-options "-O2 -fdump-rtl-combine" } */ +/* { dg-options "-O2" } */ int f1 (int x, int y) @@ -31,4 +31,4 @@ f5 (long long x, long long y) return (x & ~0xffffffffull) | (y & 0xffffffff); } -/* { dg-final { scan-rtl-dump-times "\\*aarch64_bfi" 5 "combine" } } */ +/* { dg-final { scan-assembler-times {\tbfi\t} 5 } } */