================ 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