<div dir="ltr"><div class="gmail_extra">Generally LGTM</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 6, 2015 at 6:21 PM, Simon Pilgrim <span dir="ltr"><<a href="mailto:llvm-dev@redking.me.uk" target="_blank">llvm-dev@redking.me.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":b2y" class="a3s" style="overflow:hidden">+    if (isa<ConstantSDNode>(EltNo) && VT == InVec.getValueType() &&<br>
+        VT.getScalarType() == InVal.getValueType()) {<br>
+      int Elt = cast<ConstantSDNode>(EltNo)->getZExtValue();</div></blockquote></div><br></div><div class="gmail_extra">Nit: please use a dyn_cast and a condition variable for the ConstantSDNode thing.</div></div>