Patchwork [ARM] Big Endian and Generic tuning

login
register
mail settings
Submitter Andrew Stubbs
Date Oct. 26, 2011, 1:54 p.m.
Message ID <4EA8112E.30609@codesourcery.com>
Download mbox | patch
Permalink /patch/121905/
State New
Headers show

Comments

Andrew Stubbs - Oct. 26, 2011, 1:54 p.m.
On 25/10/11 15:16, Richard Earnshaw wrote:
> The difficulty on R profile is that although BE-32 mode is obsolete,
> there is a config bit that can be set to make the CPU byte-swap the
> instruction stream to make it behave largely like it is running in BE-32
> mode.
>
> I think really we should be pushing the R-profile code towards having
> BE-8 as the default; but to do that we really need a compiler option
> that can override this behaviour (probably -mbe-[8|32]).  Once we have
> that option, we can fix the compiler to prefer BE-8 as there is then a
> way to get back the legacy behaviour.

Here's an updated patch that makes no generalizations.

OK?

Andrew
Richard Earnshaw - Oct. 26, 2011, 2:15 p.m.
On 26/10/11 14:54, Andrew Stubbs wrote:
> On 25/10/11 15:16, Richard Earnshaw wrote:
>> The difficulty on R profile is that although BE-32 mode is obsolete,
>> there is a config bit that can be set to make the CPU byte-swap the
>> instruction stream to make it behave largely like it is running in BE-32
>> mode.
>>
>> I think really we should be pushing the R-profile code towards having
>> BE-8 as the default; but to do that we really need a compiler option
>> that can override this behaviour (probably -mbe-[8|32]).  Once we have
>> that option, we can fix the compiler to prefer BE-8 as there is then a
>> way to get back the legacy behaviour.
> 
> Here's an updated patch that makes no generalizations.
> 
> OK?
> 

Yep

R.

> Andrew
> 
> 
> be.patch
> 
> 
> 2011-10-26  Andrew Stubbs  <ams@codesourcery.com>
> 
> 	gcc/
> 	* config/arm/bpabi.h (BE8_LINK_SPEC): Recognize generic-armv7 tuning.
> 
> --- a/gcc/config/arm/bpabi.h
> +++ b/gcc/config/arm/bpabi.h
> @@ -58,6 +58,7 @@
>  #define BE8_LINK_SPEC \
>    " %{mbig-endian:%{march=armv7-a|mcpu=cortex-a5	\
>     |mcpu=cortex-a8|mcpu=cortex-a9|mcpu=cortex-a15	\
> +   |mcpu=generic-armv7-a				\
>     |march=armv7-m|mcpu=cortex-m3			\
>     |march=armv7e-m|mcpu=cortex-m4			\
>     |march=armv6-m|mcpu=cortex-m0			\
Andrew Stubbs - Nov. 1, 2011, 10:01 p.m.
On 26/10/11 10:15, Richard Earnshaw wrote:
>> Here's an updated patch that makes no generalizations.
>>
>> OK?
>>
>
> Yep

Committed.

Andrew

Patch

2011-10-26  Andrew Stubbs  <ams@codesourcery.com>

	gcc/
	* config/arm/bpabi.h (BE8_LINK_SPEC): Recognize generic-armv7 tuning.

--- a/gcc/config/arm/bpabi.h
+++ b/gcc/config/arm/bpabi.h
@@ -58,6 +58,7 @@ 
 #define BE8_LINK_SPEC \
   " %{mbig-endian:%{march=armv7-a|mcpu=cortex-a5	\
    |mcpu=cortex-a8|mcpu=cortex-a9|mcpu=cortex-a15	\
+   |mcpu=generic-armv7-a				\
    |march=armv7-m|mcpu=cortex-m3			\
    |march=armv7e-m|mcpu=cortex-m4			\
    |march=armv6-m|mcpu=cortex-m0			\