On Mon, 3 Feb 2020 at 18:20, Mihail Ionescu <mihail.ione...@foss.arm.com> wrote: > > Hi, > > We noticed that the profile for armv8.1-m.main was not set in arm-cpus.in > , which led to TARGET_ARM_ARCH_PROFILE and _ARM_ARCH_PROFILE not being > defined properly. > > > > gcc/ChangeLog: > > 2020-02-03 Mihail Ionescu <mihail.ione...@arm.com> > > * config/arm/arm-cpus.in: Set profile M > for armv8.1-m.main. > > > Ok for trunk? > > Regards, > Mihail > > > ############### Attachment also inlined for ease of reply > ############### > > > diff --git a/gcc/config/arm/arm-cpus.in b/gcc/config/arm/arm-cpus.in > index > 1805b2b1cd8d6f65a967b4e3945257854a7e0fc1..96f584da325172bd1460251e2de0ad679589d312 > 100644 > --- a/gcc/config/arm/arm-cpus.in > +++ b/gcc/config/arm/arm-cpus.in > @@ -692,6 +692,7 @@ begin arch armv8.1-m.main > tune for cortex-m7 > tune flags CO_PROC > base 8M_MAIN > + profile M > isa ARMv8_1m_main > # fp => FPv5-sp-d16; fp.dp => FPv5-d16 > option dsp add armv7em >
I'm wondering whether this is obvious? OTOH, what's the impact of missing this (or why didn't we notice the problem via a failing testcase?) Christophe
