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: [PATCH] replace LIBGCC2_WORDS_BIG_ENDIAN with __WORDS_BIG_ENDIAN__


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


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