<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Feb 3, 2014 at 3:30 PM, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="im"><div class="gmail_extra"><div class="gmail_quote">On 3 February 2014 22:57, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Any reason not to just fix this in the frontend to begin with? (is it going to be substantially harder)<br></div></div></div></div></blockquote><div></div>

</div><br></div></div><div class="gmail_extra">I was not involved on the original discussion, but I wonder... If a flag is not recognized by the front-end, should it still be emitted to the back-end? I think that Clang should emit a "cpu not recognized, default to whatever cpu" warning and set the cpu to whatever default it should.</div>

<div class="gmail_extra"><br></div><div class="gmail_extra">Of course, this is orthogonal to the bug in question, which should be fixed even if the front-end stops being silly, as we do have (and can have) many front-ends.</div>
</div></blockquote><div><br>Knowing little about argument handling in LLVM, I'm not sure that's true - I expect we could replace string-identification with an enum or the like for frontend-to-LLVM communication so there'd be no need for LLVM to revalidate the input (I guess llc might still have to duplicate this checking). That'd be my abstract ideal/idea anyway... perhaps there are other issues.<br>
<br>Certainly if this does have to be fixed in LLVM anyway, we might as well do it regardless of how soon the fix comes to Clang.</div></div></div></div>