This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [ARM] pass "--be8" to linker when linking for M profile


On 15/09/11 03:41, Bin Cheng wrote:
> Hi,
> The linker should do endian swizzling at link-time according to "--be8"
> option.
> This patch modifies BE8_LINK_SPEC by adding cortex-m processors in the specs
> string.
> 
> Since R-profile supports configurable big-endian instruction fetch, I didn't
> include it here.
> 
> Is it ok? Thanks.
> 
> 2011-09-15  Cheng Bin <bin.cheng@arm.com>
>                 * config/arm/bpabi.h (BE8_LINK_SPEC): add cortex-m arch and
> processors.
> 
> Thanks-chengbin=
> 
> 
> gcc-be8-for-m-profile.patch
> 
> 

+#define BE8_LINK_SPEC  \
+  " %{mbig-endian:%{march=armv7-a|mcpu=cortex-a5 \
+   |mcpu=cortex-a8|mcpu=cortex-a9|mcpu=cortex-a15 \
+   |march=armv7-m|march=armv7e-m|mcpu=cortex-m3|mcpu=cortex-m4 \
+   |march=armv6-m|mcpu=cortex-m0:%{!r:--be8}}}"


Please sort this so that the list is ordered alphabetically by
architecture/cpu (with architectures first).

It might save some patch churn in the future if each element was put on
a line on its own.

OK with that change.

R.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]