[PATCH] D11289: [X86] Widen the 'AND' mask if doing so shrinks the encoding size

escha escha at apple.com
Fri Jul 17 09:29:04 PDT 2015


escha added inline comments.

================
Comment at: lib/Target/X86/X86ISelDAGToDAG.cpp:2237
@@ +2236,3 @@
+        // compile time.
+        if ((int8_t)Val < 0) {
+          if (NVT.SimpleTy == MVT::i64)
----------------
Is this right? (int8_t)-384 would still pass this check, right?


http://reviews.llvm.org/D11289







More information about the llvm-commits mailing list