<div dir="ltr">I don't care when it went in Craig, it went in without any review at all.<div><br></div><div>Colin, please revert this immediately and ping the thread for a proper review. You've been around long enough to know that this kind of behavior (posting for review, clearly having concerns about it, and then landing without any review happening) is completely unacceptable.</div><br><div class="gmail_quote"><div dir="ltr">On Tue, Dec 29, 2015 at 8:19 PM Craig Topper via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">craig.topper added a comment.<br>
<br>
I know this revision went in a while ago, but this seems like a really terribly performing option for Hexagon. It does a linear scan through the entire match table (which at the time of this writing is 2160 entries) for any instruction that starts with a non-token. There appear to be about ~1500 such mnemonic-less entries in table so about 3/4 of the entire table.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D14257" rel="noreferrer" target="_blank">http://reviews.llvm.org/D14257</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div></div>