This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[PATCH] rs6000: Fix pr69946.c testcase (PR85126)


After middle-end changes combine now gets fed different input, from
which it makes different (but just as efficient) code.  So remove the
test for particular asm output.

Tested, committing.


Segher


2018-04-03  Segher Boessenkool  <segher@kernel.crashing.org>

gcc/testsuite/
	PR target/85126
	* gcc.target/powerpc/pr69946: Adjust comment.  Remove
	scan-assembler-times clause.

---
 gcc/testsuite/gcc.target/powerpc/pr69946.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/gcc/testsuite/gcc.target/powerpc/pr69946.c b/gcc/testsuite/gcc.target/powerpc/pr69946.c
index eb0c365..e0ff422 100644
--- a/gcc/testsuite/gcc.target/powerpc/pr69946.c
+++ b/gcc/testsuite/gcc.target/powerpc/pr69946.c
@@ -2,9 +2,9 @@
 /* { dg-skip-if "" { powerpc_elfv2 } } */
 /* { dg-options "-O2" } */
 
-/* This generates a rotate:DI by 44, with mask 0xf00, which is implemented
-   using a rlwinm instruction.  We used to write 44 for the shift count
-   there; it should be 12.  */
+/* This used to generate a rotate:DI by 44, with mask 0xf00, which is
+   implemented using a rlwinm instruction.  We used to write 44 for the
+   shift count there; it should be 12.  */
 
 struct A
 {
@@ -35,4 +35,3 @@ foo (void)
 }
 
 /* { dg-final { scan-assembler-not {(?n)rlwinm.*,44,20,23} } } */
-/* { dg-final { scan-assembler-times {(?n)rlwinm.*,12,20,23} 1 } } */
-- 
1.8.3.1


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]