[PATCH] [InstCombine][X86] Improved folding of calls to Intrinsic::x86_sse4a_insertqi.

David Majnemer david.majnemer at gmail.com
Tue Dec 9 17:27:24 PST 2014


================
Comment at: lib/Transforms/InstCombine/InstCombineCalls.cpp:784
@@ +783,3 @@
+        // results are undefined.
+        if ((Index + Length) > 64)
+          return ReplaceInstUsesWith(CI, UndefValue::get(II->getType()));
----------------
Is there anything that prevents `Index + Length` from wrapping around? If not, what happens?

http://reviews.llvm.org/D6583






More information about the llvm-commits mailing list