[PATCH] AArch64: Add command line option to select big or little endian

[Reid Kleckner]

On Thu, Apr 3, 2014 at 12:55 AM, scott douglass <scott.douglass at arm.com>wrote:

>
>   The -EB/-EL options already exist, but effect only the MIPS backend
> (they are silently ignored for ARM target).  This patch introduces
> -mbig-endian/-mlittle-endian as aliases for them.  The options are:
>

OMG.  Those were added last year and it doesn't look like it was reviewed.
 I doubt we can remove them, but let's not encourage them.


>   [this is really just a restatement of what Tim said earlier]
>   1. leave it as it is (possibly removing the tests for -EB/-EL from ARM
> targets)
>   2. invert the aliasing (but that would mean otherwise needless changes
> to the MIPS targets)
>

Let's do this.  It's really not that big of a deal.


>   3. explicitly fault use of -EL/-EB for ARM targets (doesn't seem
> worthwhile)
>
>   Personally I vote for 1 with the -EB/-EL tests removed (or left in).
>
> http://llvm-reviews.chandlerc.com/D3215
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140403/d951daa1/attachment.html>