> -----Original Message----- > From: Gcc-patches <gcc-patches-boun...@gcc.gnu.org> On Behalf Of > Christophe Lyon via Gcc-patches > Sent: 24 February 2021 15:14 > To: gcc Patches <gcc-patches@gcc.gnu.org> > Subject: Re: arm: Fix CMSE support detection in libgcc (PR target/99157) > > On Tue, 23 Feb 2021 at 18:29, Christophe Lyon > <christophe.l...@linaro.org> wrote: > > > > As discussed in the PR, the Makefile fragment lacks a double '$' to > > get the return-code from GCC invocation, resulting is CMSE support > > missing from multilibs. > > > > I checked that the simple patch proposed in the PR fixes the problem. > > BTW, if OK for trunk, I will also push it to gcc-10. >
Ok. Thanks, Kyrill > Christophe > > > > > 2021-02-23 Christophe Lyon <christophe.l...@linaro.org> > > Hau Hsu <hsuhau...@gmail.com> > > > > PR target/99157 > > libgcc/ > > * config/arm/t-arm: Fix cmse support detection. > > > > diff --git a/libgcc/config/arm/t-arm b/libgcc/config/arm/t-arm > > index 364f40e..3625a25 100644 > > --- a/libgcc/config/arm/t-arm > > +++ b/libgcc/config/arm/t-arm > > @@ -4,7 +4,7 @@ LIB1ASMFUNCS = _thumb1_case_sqi > _thumb1_case_uqi > > _thumb1_case_shi \ > > > > HAVE_CMSE:=$(findstring __ARM_FEATURE_CMSE,$(shell > > $(gcc_compile_bare) -dM -E - </dev/null)) > > HAVE_V81M:=$(findstring armv8.1-m.main,$(gcc_compile_bare)) > > -ifeq ($(shell $(gcc_compile_bare) -E -mcmse - </dev/null >/dev/null > > 2>/dev/null; echo $?),0) > > +ifeq ($(shell $(gcc_compile_bare) -E -mcmse - </dev/null >/dev/null > > 2>/dev/null; echo $$?),0) > > CMSE_OPTS:=-mcmse > > endif
