diff mbox

[ARM] Deprecate -mwords-little-endian

Message ID g4aad0nagc.fsf@linaro.org
State New
Headers show

Commit Message

Richard Sandiford June 29, 2011, 11:28 a.m. UTC
ARM has an option called -mwords-little-endian that provides big-endian
compatibility with pre-2.8 compilers.  When I asked Richard about it,
he seemed to think it had outlived its usefulness, so this patch
deprecates it.  We can then remove it once 4.7 is out.

Tested on arm-linux-gnueabi.  OK to install?  If so, I'll do a patch
for the web page as well.

Richard

Comments

Richard Earnshaw June 29, 2011, 1:07 p.m. UTC | #1
On 29/06/11 12:28, Richard Sandiford wrote:
> ARM has an option called -mwords-little-endian that provides big-endian
> compatibility with pre-2.8 compilers.  When I asked Richard about it,
> he seemed to think it had outlived its usefulness, so this patch
> deprecates it.  We can then remove it once 4.7 is out.
> 
> Tested on arm-linux-gnueabi.  OK to install?  If so, I'll do a patch
> for the web page as well.
> 

Please also update the in-line help text in arm.opt.  OK with that change.

R.

> Richard
> 
> 
> Index: gcc/config/arm/arm.c
> ===================================================================
> --- gcc/config/arm/arm.c	2011-06-29 09:33:37.000000000 +0100
> +++ gcc/config/arm/arm.c	2011-06-29 12:20:35.000000000 +0100
> @@ -1483,6 +1483,10 @@ arm_option_override (void)
>    if (TARGET_APCS_FLOAT)
>      warning (0, "passing floating point arguments in fp regs not yet supported");
>  
> +  if (TARGET_LITTLE_WORDS)
> +    warning (OPT_Wdeprecated, "%<mwords-little-endian%> is deprecated and "
> +	     "will be removed in a future release");
> +
>    /* Initialize boolean versions of the flags, for use in the arm.md file.  */
>    arm_arch3m = (insn_flags & FL_ARCH3M) != 0;
>    arm_arch4 = (insn_flags & FL_ARCH4) != 0;
> Index: gcc/doc/invoke.texi
> ===================================================================
> --- gcc/doc/invoke.texi	2011-06-22 16:45:29.000000000 +0100
> +++ gcc/doc/invoke.texi	2011-06-29 12:17:40.000000000 +0100
> @@ -10237,7 +10237,7 @@ Generate code for a little-endian word o
>  order.  That is, a byte order of the form @samp{32107654}.  Note: this
>  option should only be used if you require compatibility with code for
>  big-endian ARM processors generated by versions of the compiler prior to
> -2.8.
> +2.8.  This option is now deprecated.
>  
>  @item -mcpu=@var{name}
>  @opindex mcpu
>
diff mbox

Patch

Index: gcc/config/arm/arm.c
===================================================================
--- gcc/config/arm/arm.c	2011-06-29 09:33:37.000000000 +0100
+++ gcc/config/arm/arm.c	2011-06-29 12:20:35.000000000 +0100
@@ -1483,6 +1483,10 @@  arm_option_override (void)
   if (TARGET_APCS_FLOAT)
     warning (0, "passing floating point arguments in fp regs not yet supported");
 
+  if (TARGET_LITTLE_WORDS)
+    warning (OPT_Wdeprecated, "%<mwords-little-endian%> is deprecated and "
+	     "will be removed in a future release");
+
   /* Initialize boolean versions of the flags, for use in the arm.md file.  */
   arm_arch3m = (insn_flags & FL_ARCH3M) != 0;
   arm_arch4 = (insn_flags & FL_ARCH4) != 0;
Index: gcc/doc/invoke.texi
===================================================================
--- gcc/doc/invoke.texi	2011-06-22 16:45:29.000000000 +0100
+++ gcc/doc/invoke.texi	2011-06-29 12:17:40.000000000 +0100
@@ -10237,7 +10237,7 @@  Generate code for a little-endian word o
 order.  That is, a byte order of the form @samp{32107654}.  Note: this
 option should only be used if you require compatibility with code for
 big-endian ARM processors generated by versions of the compiler prior to
-2.8.
+2.8.  This option is now deprecated.
 
 @item -mcpu=@var{name}
 @opindex mcpu