[cfe-commits] [PATCH] Add -ldl for libgcc to follow Android ABI

[Logan Chien]

Dear all,

  There's the revised patch.  Rebasing on the ANDROIDEABI to Android
renaming CL.
Please have a look.  Thanks.

Sincerely,
Logan

For your convenience, there's the online diff:
https://github.com/loganchien/clang-logan/commit/b539e90ec629bdfcf40657251f3610eec2cc949f


On Wed, Aug 29, 2012 at 5:50 PM, Logan Chien <tzuhsiang.chien at gmail.com>wrote:

> No, I don't have commit access yet.  May you commit it for me?  Thanks.
>
> Logan
>
>
>
> On Wed, Aug 29, 2012 at 3:46 PM, Chandler Carruth <chandlerc at google.com>wrote:
>
>> LGTM, do you have commit access?
>>
>>
>> On Wed, Aug 29, 2012 at 12:28 AM, Logan Chien <tzuhsiang.chien at gmail.com>wrote:
>>
>>> Hi,
>>>
>>>   According to Android ABI, we have to link with libdl.so, if "-static"
>>> flag is not set.  Otherwise, we will get link error on MIPS architecture,
>>> because its libgcc references `_Unwind_Find_FDE' and `dl_iterate_phdr',
>>> but these symbols are defined in libdl.so.
>>>
>>>   I hope the patch can resolve this issue.  And the test cases has been
>>> attached.  Please have a look.  Thanks.
>>>
>>> Sincerely,
>>> Logan
>>>
>>>
>>> _______________________________________________
>>> cfe-commits mailing list
>>> cfe-commits at cs.uiuc.edu
>>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>>>
>>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120831/172adb16/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-Add-ldl-for-lgcc-in-Android-environment.patch
Type: application/octet-stream
Size: 5024 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120831/172adb16/attachment.obj>