<p dir="ltr">I don't think clang's code is currently good enough in that area to matter much. We need a refactoring around target description anyway, and it won't be now. :-) </p>
<p dir="ltr">LGTM. </p>
<p dir="ltr">Cheers, <br>
Renato </p>
<div class="gmail_quote">On 13 May 2013 23:18, "Rafael Espíndola" <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 13 May 2013 18:02, JF Bastien <<a href="mailto:jfb@google.com">jfb@google.com</a>> wrote:<br>
> I don't feel strongly about it (and don't know the codebase much) but<br>
> there's similar code here:<br>
>   lib/Target/ARM/MCTargetDesc/ARMMCTargetDesc.cpp<br>
><br>
> I'll defer to those who know LLVM better than I do for this, and it<br>
> otherwise looks good to me.<br>
<br>
Thanks for the pointer, but I don't think it would help a lot in this<br>
case. The version is still returned as "+v7" or "+v6". I assume at<br>
some point a "+v8" will be added, so we would still have to parse it<br>
on our end.<br>
<br>
Renato, what do you think?<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div>