[cfe-dev] [PATCH] openSUSE toolchain update

[Chris Lattner]

Can a redhat user test this?

-Chris

On May 22, 2011, at 9:53 AM, İsmail Dönmez wrote:

> Thanks, here is one for the Redhat typo I mentioned.
> 
> Regards.
> 
> On Sun, May 22, 2011 at 6:50 PM, Chris Lattner <clattner at apple.com> wrote:
> Applied in r131857, thanks
> 
> On May 22, 2011, at 9:40 AM, İsmail Dönmez wrote:
> 
>> Hi Chris;
>> 
>> On Sun, May 22, 2011 at 6:32 PM, Chris Lattner <clattner at apple.com> wrote:
>> Hi Ismail,
>> 
>> This doesn't apply cleanly to mainline.  The changes required look trivial, but I'd prefer you to do it because I can't test this.
>> 
>> New patch attached, but this line from Toolchains.cpp:
>> 
>>   if (IsRedhat(Distro))
>>     ExtraOpts.push_back("--no-add-needed");
>> 
>> looks like a typo, correct option is "--no-as-needed".
>> 
>> Regards,
>> ismail
>>  
>> <toolchain-update.patch>
> 
> 
> <redhat-typo.patch>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20110522/d71b1c62/attachment.html>