<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Apr 3, 2014 at 12:55 AM, scott douglass <span dir="ltr"><<a href="mailto:scott.douglass@arm.com" target="_blank">scott.douglass@arm.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  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:<br></blockquote>
<div><br></div><div>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.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

  [this is really just a restatement of what Tim said earlier]<br>
  1. leave it as it is (possibly removing the tests for -EB/-EL from ARM targets)<br>
  2. invert the aliasing (but that would mean otherwise needless changes to the MIPS targets)<br></blockquote><div><br></div><div>Let's do this.  It's really not that big of a deal.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

  3. explicitly fault use of -EL/-EB for ARM targets (doesn't seem worthwhile)<br>
<br>
  Personally I vote for 1 with the -EB/-EL tests removed (or left in).<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D3215" target="_blank">http://llvm-reviews.chandlerc.com/D3215</a><br>
</blockquote></div><br></div></div>