[cfe-dev] [PATCH] openSUSE toolchain update

[Chris Lattner]

ok thanks.

-Chris

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

> Oh nm, its not a typo. Here comes the ld(1):
> 
> --no-add-needed 
> 
> This option affects the treatment of dynamic libraries from ELF DT_NEEDED tags in dynamic libraries mentioned on the command line after the --no-add-needed option. Normally, the linker will add a DT_NEEDED tag for each dynamic library from DT_NEEDED tags. --no-add-needed causes DT_NEEDED tags will never be emitted for thoselibraries from DT_NEEDED tags. --add-needed restores the default behaviour.
> 
> Sorry! ;)
> 
> On Sun, May 22, 2011 at 6:55 PM, Chris Lattner <clattner at apple.com> wrote:
> 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/8881056a/attachment.html>