<br><br><div class="gmail_quote">On Tue Nov 04 2014 at 5:26:00 AM Renato Golin <<a href="mailto:renato.golin@linaro.org">renato.golin@linaro.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Eric,<br>
<br>
>From my point of view, the issue this solves is that now you can say "-triple armvelocipede" and it'll work. That's far from ideal.<br>
<br></blockquote><div><br></div><div>*shrug* You'll get a generic arm processor. I agree that it's not ideal.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I'd prefer to have a proper parser to get arch+cpu+features from triple/march/mcpu/mfpu/mtune options, but that'll require the TargetInfo parser, which will take a while. For now, this patch would at least cover the bogus cases in ARM.<br>
<br></blockquote><div><br></div><div>If it's a problem you're running into sure.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I agree some tests are in order, but I believe we already have some.<br>
<br></blockquote><div><br></div><div>It will need negative tests for certain. I'd prefer to get all of them squirreled away.</div><div><br></div><div>-eric</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Gabor,<br>
<br>
Have you checked for current tests on that? Are they complete? Even if they are, we should have at least a few negative tests ("RUN: not" lines) to make sure we don't accept the things we say we won't.<br>
<br>
cheers,<br>
--renato<br>
<br>
<a href="http://reviews.llvm.org/D6058" target="_blank">http://reviews.llvm.org/D6058</a><br>
<br>
<br>
</blockquote></div>