<div dir="ltr">The inconsistency is clearly troubling and is worth fixing. This review may have gotten off to a bad start by focusing on the performance aspect. Any time correctness and performance get mixed into the same discussion this sort of thing tends to happen.<div><br></div><div>(also, I don't think anybody is really advocating the use of string comparisons)</div><div><br></div><div>@vedant: do you know if it is important for compatibility for us to use strings in the tests Fiona pointed out? Can we just switch those tests to use a real intrinsic?<br><div><div><br></div><div>-- Sean Silva</div></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 5, 2015 at 5:40 PM, escha <span dir="ltr"><<a href="mailto:escha@apple.com" target="_blank">escha@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">escha abandoned this revision.<br>
escha added a comment.<br>
<br>
Since people don't seem to be particularly interested in fixing broken+inconsistent behavior and believe performing string comparisons is a reasonable and performant way of checking class membership, I'm not going to bother with this.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D13427" rel="noreferrer" target="_blank">http://reviews.llvm.org/D13427</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br></div>