diff mbox series

arm: remove error in CPP_SPEC when -mlittle-endian and -mbig-endian are used together

Message ID 20210512085142.31934-1-christophe.lyon@linaro.org
State New
Headers show
Series arm: remove error in CPP_SPEC when -mlittle-endian and -mbig-endian are used together | expand

Commit Message

Christophe Lyon May 12, 2021, 8:51 a.m. UTC
arm.h has had this error message since 1997, but it is no longer
needed since option parsing has been improved: -mXXX-endian is handled
via arm.opt and updates the BIG_END mask. So, the last
instance of -mXXX-endian on the command line wins.

Tested on many arm* configurations, with no impact on the testsuite results.

2021-05-12  Christophe Lyon  <christophe.lyon@linaro.org>

	gcc/
	* config/arm/arm.h (CPP_SPEC): Remove error message about
	-mlittle-endian/-mbig-endian conflict.
---
 gcc/config/arm/arm.h | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

Comments

Richard Earnshaw May 12, 2021, 10:33 a.m. UTC | #1
On 12/05/2021 09:51, Christophe Lyon via Gcc-patches wrote:
> arm.h has had this error message since 1997, but it is no longer
> needed since option parsing has been improved: -mXXX-endian is handled
> via arm.opt and updates the BIG_END mask. So, the last
> instance of -mXXX-endian on the command line wins.
> 
> Tested on many arm* configurations, with no impact on the testsuite results.
> 
> 2021-05-12  Christophe Lyon  <christophe.lyon@linaro.org>
> 
> 	gcc/
> 	* config/arm/arm.h (CPP_SPEC): Remove error message about
> 	-mlittle-endian/-mbig-endian conflict.

OK.

R.

> ---
>   gcc/config/arm/arm.h | 4 +---
>   1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/gcc/config/arm/arm.h b/gcc/config/arm/arm.h
> index e430e4d50ed..8e5bd579323 100644
> --- a/gcc/config/arm/arm.h
> +++ b/gcc/config/arm/arm.h
> @@ -89,9 +89,7 @@ extern tree arm_bf16_ptr_type_node;
>   
>   
>   #undef  CPP_SPEC
> -#define CPP_SPEC "%(subtarget_cpp_spec)					\
> -%{mbig-endian:%{mlittle-endian:						\
> -	%e-mbig-endian and -mlittle-endian may not be used together}}"
> +#define CPP_SPEC "%(subtarget_cpp_spec)"
>   
>   #ifndef CC1_SPEC
>   #define CC1_SPEC ""
>
diff mbox series

Patch

diff --git a/gcc/config/arm/arm.h b/gcc/config/arm/arm.h
index e430e4d50ed..8e5bd579323 100644
--- a/gcc/config/arm/arm.h
+++ b/gcc/config/arm/arm.h
@@ -89,9 +89,7 @@  extern tree arm_bf16_ptr_type_node;
 
 
 #undef  CPP_SPEC
-#define CPP_SPEC "%(subtarget_cpp_spec)					\
-%{mbig-endian:%{mlittle-endian:						\
-	%e-mbig-endian and -mlittle-endian may not be used together}}"
+#define CPP_SPEC "%(subtarget_cpp_spec)"
 
 #ifndef CC1_SPEC
 #define CC1_SPEC ""