Hi, Here attached the second version patch, with changes mentioned previously.
Is it ok?
Thanks-chengbin
2011-09-16 Cheng Bin <bin.ch...@arm.com>
* config/arm/bpabi.h (BE8_LINK_SPEC): Add cortex-m arch and
processors.
> -----Original Message-----
> From: Richard Earnshaw
> Sent: Thursday, September 15, 2011 6:46 PM
> To: Bin Cheng
> Cc: gcc-patches@gcc.gnu.org
> Subject: 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.ch...@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.
gcc-be8-for-m-profile-20110916.patch
Description: Binary data
