Hi Steve,
On 26/09/18 15:44, Steve Ellcey wrote:
The patch for PR rtl-optimization/85160 which allowed combine to convert
two instructions into two different instructions if they had a lower cost
caused a couple of regressions on aarch64. This patch fixes one of them.
After the above patch, the gcc.target/aarch64/ashltidisi.c test generated
3 asr instructions instead of 4. Given that the overall test now has
two fewer instructions and appears to be superior to the original generated
code, this patch just updates the test to reflect the newly generated code.
Tested on aarch64, OK for checkin?
Steve Ellcey
sell...@cavium.com
2018-09-26 Steve Ellcey <sell...@cavium.com>
* gcc.target/aarch64/ashltidisi.c: Expect 3 asr instructions
instead of 4.
diff --git a/gcc/testsuite/gcc.target/aarch64/ashltidisi.c
b/gcc/testsuite/gcc.target/aarch64/ashltidisi.c
index 293a0f2..e2a0997 100644
--- a/gcc/testsuite/gcc.target/aarch64/ashltidisi.c
+++ b/gcc/testsuite/gcc.target/aarch64/ashltidisi.c
@@ -45,5 +45,5 @@ main (int argc, char **argv)
return 0;
}
-/* { dg-final { scan-assembler-times "asr" 4 } } */
+/* { dg-final { scan-assembler-times "asr" 3 } } */
/* { dg-final { scan-assembler-not "extr\t" } } */
This looks obvious to me.
Thanks,
Kyrill
