CLANG special case

Jim Laskey (Oracle) james.laskey at oracle.com
Tue Oct 6 12:30:48 UTC 2015


I’ve updated to El Capitan and, of course, builds fail, and, of course, I modify hotspot/make/bsd/makefiles/gcc.make one more time and…   I think this conditional clause should be removed at the very least (commenting to indicate needs investigation), or someone should research and see which version of clang fixes the issues associate with the patch.  Since it’s likely that no one has the cycles, please remove the condition.

Cheers,

— Jim



diff -r a02911828e48 make/bsd/makefiles/gcc.make
--- a/make/bsd/makefiles/gcc.make	Wed Sep 30 07:41:36 2015 -0700
+++ b/make/bsd/makefiles/gcc.make	Tue Oct 06 09:22:50 2015 -0300
@@ -313,21 +313,13 @@
 
 # Work around some compiler bugs.
 ifeq ($(USE_CLANG), true)
-  # Clang <= 6.1
-  ifeq ($(shell expr \
-      $(CC_VER_MAJOR) \< 6 \| \
-      \( $(CC_VER_MAJOR) = 6 \& $(CC_VER_MINOR) \<= 1 \) \
-    ), 1)
-    OPT_CFLAGS/loopTransform.o += $(OPT_CFLAGS/NOOPT)
-    OPT_CFLAGS/unsafe.o += -O1
-  else
-    $(error "Update compiler workarounds for Clang $(CC_VER_MAJOR).$(CC_VER_MINOR)")
-  endif
+  OPT_CFLAGS/loopTransform.o += $(OPT_CFLAGS/NOOPT)
+  OPT_CFLAGS/unsafe.o += -O1
 else
   # 6835796. Problem in GCC 4.3.0 with mulnode.o optimized compilation.
   ifeq ($(shell expr $(CC_VER_MAJOR) = 4 \& $(CC_VER_MINOR) = 3), 1)
     OPT_CFLAGS/mulnode.o += $(OPT_CFLAGS/NOOPT)
-  endif 
+  endif
 endif
 
 # Flags for generating make dependency flags.




More information about the build-dev mailing list