[PATCH] D30213: [X86][SSE] Enable post-legalize vXi64 shuffle combining on 32-bit targets

Andrea Di Biagio via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 21 10:57:30 PST 2017


andreadb added inline comments.


================
Comment at: test/CodeGen/X86/dagcombine-cse.ll:1-2
 ; REQUIRES: asserts
-; RUN: llc < %s -march=x86 -mattr=+sse2 -mtriple=i386-apple-darwin -stats 2>&1 | grep asm-printer | grep 13
+; RUN: llc < %s -march=x86 -mattr=+sse2 -mtriple=i386-apple-darwin -stats 2>&1 | grep asm-printer | grep 12
 
----------------
Do you think that there is still value in keeping this test?

If we want to keep this test, then I suggest to rewrite the RUN line (i.e. use FileCheck instead of grep, and regenerate the test using the update script).


Repository:
  rL LLVM

https://reviews.llvm.org/D30213





More information about the llvm-commits mailing list