Re: PATCH: PR bootstrap/54209: [4.8 Regression] Failed to build gcc for Android/x86

["H.J. Lu" <hjl dot tools at gmail dot com>]

On Tue, Aug 14, 2012 at 12:38 PM, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Thu, Aug 9, 2012 at 3:17 PM, Ian Lance Taylor <iant@google.com> wrote:
>> On Thu, Aug 9, 2012 at 9:39 AM, H.J. Lu <hongjiu.lu@intel.com> wrote:
>>>
>>> Bionic C library doesn't provide link.h.
>>
>> Does Bionic provide dl_iterate_phdr?  If it does, I'll just note in
>> passing that it would be straightforward to simply incorporate the
>> required types and constants in unwind-dw2-fde-dip.c directly, and
>> avoid the #include.  If it doesn't, then of course nothing will make
>> this code work correctly.
>>
>
> dl_iterate_phdr is provided in libdl.so, which is always linked with
> dynamic executables:
>
> #define ANDROID_LIB_SPEC \
>   "%{!static: -ldl}"
>
>
> This patch fixes Android/x86 build on trunk. OK to install?
>
> Thanks.
>
> --
> H.J.
> ----
> 2012-08-14  H.J. Lu  <hongjiu.lu@intel.com>
>
>         PR bootstrap/54209
>         * unwind-dw2-fde-dip.c (dl_phdr_info): New struct for Bionic C
>         library.
>         (ElfW): New macro for Bionic C library.
>         Don't include <link.h> for Bionic C library.

Wrong patch.  Here is the right one.

-- 
H.J.

Attachment: gcc-pr54209.patch
Description: Binary data