<div dir="ltr"><div>To be honest I don't know if I know the code well enough to know if that can fail. Just seemed bad to blindly assume there was only one type there. There are quite a few such "unhandled" asserts in this file so I just wrote the same. <br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 4, 2015 at 9:59 PM, <a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a> <span dir="ltr"><<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">LGTM, thanks!<br>
<br>
<br>
================<br>
Comment at: utils/TableGen/CodeGenDAGPatterns.cpp:3058<br>
@@ +3057,3 @@<br>
+    for (unsigned i = 0; i != NumResults; ++i) {<br>
+      assert(ResNodes[i]->getNumTypes() == 1 && "FIXME: Unhandled");<br>
+      ResultPattern->setType(i, ResNodes[i]->getExtType(0));<br>
----------------<br>
Please improve the string in this assert to explain what is not handled.<br>
<br>
<a href="http://reviews.llvm.org/D8073" target="_blank">http://reviews.llvm.org/D8073</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">~Craig</div>
</div>