[U-Boot] [PATCH] sh: only add -mno-fdpic if the compiler supports it

Not all SuperH toolchains support -mno-fdpic. Chances are good that if the flag doesn't work, it isn't defaulting to the FDPIC ABI, so the flag isn't needed. So only add it if it is actually supported.
Signed-off-by: Mike Frysinger vapier@gentoo.org --- arch/sh/cpu/sh2/config.mk | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/arch/sh/cpu/sh2/config.mk b/arch/sh/cpu/sh2/config.mk index f46b38f..bdd3315 100644 --- a/arch/sh/cpu/sh2/config.mk +++ b/arch/sh/cpu/sh2/config.mk @@ -24,10 +24,11 @@ ENDIANNESS += -EB
ifdef CONFIG_SH2A -PLATFORM_CPPFLAGS += -m2a -m2a-nofpu -mb -mno-fdpic -ffreestanding +PLATFORM_CPPFLAGS += -m2a -m2a-nofpu -mb -ffreestanding else # SH2 PLATFORM_CPPFLAGS += -m3e -mb endif +PLATFORM_CPPFLAGS += $(call cc-option,-mno-fdpic)
PLATFORM_RELFLAGS += -ffixed-r13 PLATFORM_LDFLAGS += $(ENDIANNESS)

Hi Mike,
Mike Frysinger wrote on 13/10/2011 05:45:47:
Not all SuperH toolchains support -mno-fdpic. Chances are good that if the flag doesn't work, it isn't defaulting to the FDPIC ABI, so the flag isn't needed. So only add it if it is actually supported.
Signed-off-by: Mike Frysinger vapier@gentoo.org
Tested on SH7264 with CodeSourcery 2011.03 SH2A toolchain.
Thanks Phil

Applied. Thanks.
Nobuhiro
2011/10/13 phil.edworthy@renesas.com:
Hi Mike,
Mike Frysinger wrote on 13/10/2011 05:45:47:
Not all SuperH toolchains support -mno-fdpic. Chances are good that if the flag doesn't work, it isn't defaulting to the FDPIC ABI, so the flag isn't needed. So only add it if it is actually supported.
Signed-off-by: Mike Frysinger vapier@gentoo.org
Tested on SH7264 with CodeSourcery 2011.03 SH2A toolchain.
Thanks Phil
participants (3)
-
Mike Frysinger
-
Nobuhiro Iwamatsu
-
phil.edworthy@renesas.com