<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Aug 5, 2013, at 9:51 AM, Tom Stellard <<a href="mailto:tom@stellard.net">tom@stellard.net</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">On Thu, Aug 01, 2013 at 07:34:30AM -0700, Tom Stellard wrote:<br><blockquote type="cite">On Wed, Jul 31, 2013 at 10:09:03PM -0700, Owen Anderson wrote:<br><blockquote type="cite"><br>On Jul 31, 2013, at 7:25 PM, Tom Stellard <<a href="mailto:tom@stellard.net">tom@stellard.net</a>> wrote:<br><br><blockquote type="cite"><blockquote type="cite"><br>The first version of this patch that I wrote:<br><a href="http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20130715/181799.html">http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20130715/181799.html</a><br>added TargetLowering::getVectorIdxTy() which targets could use to specify<br>whatever type the wanted. What do you think of using the version one patch<br>instead?<br><br></blockquote><br>Ping.<br></blockquote><br>FWIW, I prefer the approach using a target-specified type.<br><br></blockquote><br>Thanks Owen,<br><br>Do you any comments on my original patch?<br><br><a href="http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20130715/181799.html">http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20130715/181799.html</a><br><br></blockquote></blockquote></div><br><div><br></div><div><blockquote type="cite"><pre style="white-space: pre-wrap; ">- SDValue Idx = ZExtPromotedInteger(N->getOperand(2));
+ SDValue Idx = DAG.getZExtOrTrunc(N->getOperand(2), SDLoc(N), MVT::i32);</pre></blockquote><div>Should this be a parameterized type rather than i32?</div></div><div><br></div><div>--Owen</div></body></html>