This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [lto][patch] Add a program to read IL symbol tables

From: "Rafael Espindola" <espindola at google dot com>
To: "Joseph S. Myers" <joseph at codesourcery dot com>
Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>, "Diego Novillo" <dnovillo at google dot com>
Date: Wed, 20 Aug 2008 16:31:10 +0100
Subject: Re: [lto][patch] Add a program to read IL symbol tables
References: <38a0d8450808200729q14187761wcd7ed5b8e3ca7a3d@mail.gmail.com> <Pine.LNX.4.64.0808201443350.30074@digraph.polyomino.org.uk> <38a0d8450808200812r281d38f0oebe58578fb253acf@mail.gmail.com>

>> The plug-in should also share the same logic for finding libelf that
>> gcc/configure.ac has - allowing for possible different names for the
>> libelf header, and, should a --with-libelf option be added in future as
>> per <http://gcc.gnu.org/ml/gcc/2007-09/msg00117.html>, honouring that.
>> Thus I recommend putting that logic in a shared macro in config/, and
>> making sure the C sources allow for the different header names.
>
> Will work on that.

The attached patch fixes it. Is it OK?

Cheers,
-- 
Rafael Avila de Espindola

Google Ireland Ltd.
Gordon House
Barrow Street
Dublin 4
Ireland

Registered in Dublin, Ireland
Registration Number: 368047

Attachment: symtab.patch.bz2
Description: BZip2 compressed data

Follow-Ups:
- Re: [lto][patch] Add a program to read IL symbol tables
  - From: Diego Novillo

References:
- [lto][patch] Add a program to read IL symbol tables
  - From: Rafael Espindola
- Re: [lto][patch] Add a program to read IL symbol tables
  - From: Joseph S. Myers
- Re: [lto][patch] Add a program to read IL symbol tables
  - From: Rafael Espindola

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]