7.4.0: Makefile.in: ASM_CPP_OPTIONS is gcc specific about bdwgc HOT 4 CLOSED

Hi,
Could you prepare a patch for configure files?
Thanks
Regards,

Ivan

Friday, 09 May 2014, 14:49 +04:00 from marksolaris [email protected]:
The Makefile.in hardcodes the ASM_CPP_OPTIONS to 'extern -Wp,-P -x assembler-with-cpp'. These args don't work with the Sun Studio compiler.
/usr/local/bin/bash ./libtool  --tag=CC   --mode=compile cc -DHAVE_CONFIG_H   -I./include -I./include -I./libatomic_ops/src -I./libatomic_ops/src -I/usr/local/include  -I/usr/local/include -DSCM_API=extern -Wp,-P -x assembler-with-cpp -c src/sparc_mach_dep.S
libtool: compile:  cc -DHAVE_CONFIG_H -I./include -I./include -I./libatomic_ops/src -I./libatomic_ops/src -I/usr/local/include -I/usr/local/include -DSCM_API=extern -Wp,-P -x assembler-with-cpp -c src/sparc_mach_dep.S  -KPIC -DPIC -o .libs/sparc_mach_dep.o
cc: Warning: illegal option -xassembler-with-cpp
I use this as a work-around when compiling to clear the variable:
perl -pe 's#ASM_CPP_OPTIONS =.*#ASM_CPP_OPTIONS =#' -i Makefile.in
I see you already fixed the src/sparc_mach_dep.lo problem :)
—
Reply to this email directly or  view it on GitHub .

from bdwgc.

Not the configure files, but the diff for the Makefile.in is:

diff -ru Makefile.in.orig Makefile.in
--- Makefile.in.orig    Mon May 12 10:36:19 2014
+++ Makefile.in Mon May 12 10:36:25 2014
@@ -1021,7 +1021,7 @@
 # Misc
 # ----
 AM_CXXFLAGS = @GC_CFLAGS@
-@COMPILER_XLC_FALSE@ASM_CPP_OPTIONS = -Wp,-P -x assembler-with-cpp
+@COMPILER_XLC_FALSE@ASM_CPP_OPTIONS =

 #all_objs = @addobjs@ $(libgc_la_OBJECTS)
 #$(all_objs) : include/private/gcconfig.h include/private/gc_priv.h \

from bdwgc.

Makefile.in is not in the repository. The correct solution could be: test whether -x assembler-with-cpp supported and define X_ASM_WITH_CPP_SUPPORTED am macro accordingly in configure, and test the macro in Makefile.am

from bdwgc.

Should be fixed by commits 86399b5 and 6253914. Please check it.

from bdwgc.