This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] replace LIBGCC2_WORDS_BIG_ENDIAN with __WORDS_BIG_ENDIAN__
- From: Andrew Pinski <pinskia at gmail dot com>
- To: Nathan Froyd <froydnj at codesourcery dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Tue, 12 Oct 2010 13:52:56 -0700
- Subject: Re: [PATCH] replace LIBGCC2_WORDS_BIG_ENDIAN with __WORDS_BIG_ENDIAN__
- References: <20101012204838.GE24720@nightcrawler>
On Tue, Oct 12, 2010 at 1:48 PM, Nathan Froyd <froydnj@codesourcery.com> wrote:
> The patch below eliminates the need for LIBGCC2_WORDS_BIG_ENDIAN by
> having the compiler define __WORDS_BIG_ENDIAN__ and appropriate places
> use that instead. ÂThe motivating idea here is that the compiler should
> be queried for all target-specific knowledge when compiling target
> libraries. ÂI plan to replace several other macros in a similar fashion.
>
> Boostrapped on x86_64-unknown-linux-gnu, also tested with cross to
> mips-elf and powerpc-eabispe as bi-endian targets. ÂNeed C approval for
> the cppbuiltin.c change and global approval for CPU changes etc. ÂOK to
> commit?
I like this patch as it removes one more host vs target header issue.
That is it removes one more place where we need to include a header
from gcc/config/ in a target library.
Thanks,
Andrew Pinski