Ideally hotspot would review this, not build.
so it would be helpful if hotspot found an engineer to own the bug :-
https://bugs.openjdk.java.net/browse/JDK-8138820
So far as I know this is not tracked under any other bug id.
-phil.
On 10/06/2015 05:30 AM, Jim Laskey (Oracle) wrote:
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.