[build] Use -maccumulate-outgoing-args if required by gcc

Current versions of gcc require -maccumulate-outgoing-args if any
sysv_abi functions call ms_abi functions.  This requirement is likely
to be lifted in future gcc versions, so test explicitly to see if the
current version of gcc requires -maccumulate-outgoing-args.

This problem is currently masked since the implied
-fasynchronous-unwind-tables (which is the default in current gcc
versions) implies -maccumulate-outgoing-args.

Signed-off-by: Michael Brown <mcb30@ipxe.org>

diff --git a/src/arch/x86_64/Makefile b/src/arch/x86_64/Makefile
index d2c2ff53d91dc4d2e8517583e734baddbff50d5e..b687f3407970eba6fd343810125ecf42fc7d05c1 100644 (file)
--- a/src/arch/x86_64/Makefile
+++ b/src/arch/x86_64/Makefile
@@ -26,6 +26,18 @@ CFLAGS               += -fshort-wchar
 #
 CFLAGS                 += -Ui386
 
+# Add -maccumulate-outgoing-args if required by this version of gcc
+#
+ifeq ($(CCTYPE),gcc)
+MS_ABI_TEST_CODE := extern void __attribute__ (( ms_abi )) ms_abi(); \
+                   void sysv_abi ( void ) { ms_abi(); }
+MS_ABI_TEST = $(ECHO) '$(MS_ABI_TEST_CODE)' | \
+             $(CC) -m64 -mno-accumulate-outgoing-args -x c -c - -o /dev/null \
+                   >/dev/null 2>&1
+MS_ABI_FLAGS := $(shell $(MS_ABI_TEST) || $(ECHO) '-maccumulate-outgoing-args')
+WORKAROUND_CFLAGS += $(MS_ABI_FLAGS)
+endif
+
 # x86_64-specific directories containing source files
 #
 SRCDIRS                += arch/x86_64/prefix