This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [google][4.6]Bug fix to function reordering plugin to check presence of elf.h
- From: Xinliang David Li <davidxl at google dot com>
- To: Sriraman Tallam <tmsriram at google dot com>
- Cc: Jing Yu <jingyu at google dot com>, Paul Pluzhnikov <ppluzhnikov at google dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Fri, 24 Feb 2012 23:18:32 -0800
- Subject: Re: [google][4.6]Bug fix to function reordering plugin to check presence of elf.h
- Authentication-results: mr.google.com; spf=pass (google.com: domain of davidxl@google.com designates 10.112.23.104 as permitted sender) smtp.mail=davidxl@google.com; dkim=pass header.i=davidxl@google.com
- References: <CAAs8HmzQBim4ea3fQ5TvXwTwcnQokLKLUo2yf8FR3xHqAD3fDw@mail.gmail.com>
ok.
David
On Fri, Feb 24, 2012 at 4:19 PM, Sriraman Tallam <tmsriram@google.com> wrote:
> function_reordering_plugin.c includes <elf.h> which is not available
> on non-ELF platforms building a cross-compiler. This patch checks for
> <elf.h> before including it. Otherwise, it redefines the macros used.
> This is safe because the macros will not change.
>
> For context, this linker plugin itself is only available in the google
> 4_6 branch and I will port it to other branches and make it available
> for review for trunk soon.
>
>
> 2012-02-24 ?Sriraman Tallam ?<tmsriram@google.com>
>
> ? ? ? ?* function_reordering_plugin.c: Check for presence of elf.h.
> ? ? ? ?Otherwise, redefine the elf macros used.
>
> Ok to commit?
>
> Thanks,
> -Sri.