Dan, > + bool NewUndefElts = false; > + for (unsigned i = 0; i < VWidth; i++) { > + unsigned MaskVal = Shuffle->getMaskValue(i); > + if (MaskVal == -1) { This line is warning about comparison between signed and unsigned. -bw