================
Comment at: lib/CodeGen/CGExprScalar.cpp:1224
@@ -1219,1 +1223,3 @@
+                   "Index operand too large for shufflevector mask!");
+            Args.push_back(Builder.getInt32(C->getZExtValue()));
             Args.resize(ResElts, llvm::UndefValue::get(CGF.Int32Ty));
----------------
The comment can just be "shufflemask must use an i32".  It'd be nice if you 
could avoid re-uniquing the constant when it already is an i32.


http://reviews.llvm.org/D10838




_______________________________________________
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to