[PATCH] XCore target add __LITTLE_ENDIAN__ defined macro

Robert Lytton robert at xmos.com
Fri Feb 14 06:58:52 PST 2014


Shall I commit this change independently of any decision on making it standard for all targets?
Robert

________________________________________
From: Rafael EspĂ­ndola [rafael.espindola at gmail.com]
Sent: 13 February 2014 13:04
To: Richard Osborne
Cc: reviews+D2760+public+b493f8301bfe0345 at llvm-reviews.chandlerc.com; Robert Lytton; llvm-commits; Joerg Sonnenberger
Subject: Re: [PATCH] XCore target add __LITTLE_ENDIAN__ defined macro

On 13 February 2014 05:43, Richard Osborne <richard at xmos.com> wrote:
> Hi Robert,
>
> It's hard to review this change without knowing a bit more context. In
> particular:
>
> * Is there some software in particular that relies on this macro?
> * Is this macro a standard macro? Do other targets define this macro?

It is very common. Maybe we should just define it in a central place
like we do with __BYTE_ORDER__?

CCing Joerg that might know if any architecture (other than pdp-11)
would not want __LITTLE_ENDIAN__ or __BIG_ENDIAN__ defined.

Cheers,
Rafael




More information about the llvm-commits mailing list