Orthogonal cleanups such as -ccc-clang-archs removal should be in a separate patch generally speaking. I'll go ahead and apply that part to save a round trip.<div><br></div><div>The test cases though look much like the original test cases in the patch I cleaned up for you. I had to edit those test cases significantly to get them to pass on Linux. Could you try to write the assertions in a manner more similar to the ones I ended up checking in? Specifically there is no reason to check every argument to the linker when you're really trying to isolate and test a single feature. The way you've written in the tests are simply too brittle.</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Oct 16, 2012 at 7:37 PM, David Hill <span dir="ltr"><<a href="mailto:dhill@mindcry.org" target="_blank">dhill@mindcry.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
How is this?<br>
<div class="HOEnZb"><div class="h5"><br>
On Tue, Oct 16, 2012 at 05:24:59PM -0700, Eli Friedman wrote:<br>
> On Mon, Oct 15, 2012 at 9:00 AM, David Hill <<a href="mailto:dhill@mindcry.org">dhill@mindcry.org</a>> wrote:<br>
> > Ping.  Can someone commit this?<br>
><br>
> +    if (Args.hasArg(options::OPT_pthread)) {<br>
> +      if (!Args.hasArg(options::OPT_shared) &&<br>
> +          Args.hasArg(options::OPT_pg))<br>
><br>
> I'd like to see a driver test where the check for -shared matters.<br>
><br>
> -Eli<br>
</div></div><br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>