-
Enhancement
-
Resolution: Fixed
-
P2
-
8
-
b01
-
os_x
-
Not verified
Issue | Fix Version | Assignee | Priority | Status | Resolution | Resolved In Build |
---|---|---|---|---|---|---|
JDK-8248934 | 8u271 | David Buck | P2 | Resolved | Fixed | b01 |
JDK-8244874 | 8u251 | David Buck | P2 | Resolved | Fixed | b34 |
JDK-8251612 | emb-8u271 | David Buck | P2 | Resolved | Fixed | team |
JDK-8246868 | emb-8u261 | David Buck | P2 | Resolved | Fixed | team |
In hotspot/make/bsd/makefiles/gcc.make, we check the clang version to decide whether to apply the optimization bug [0] workaround. Doing it this way forces us to repeatedly update the build file every time we use a new toolchain version. As part of the new HotSpot build system (see jdk8152666), this check was finally removed in JDK 9. While it would be inappropriate to try and backport the entire new build system to JDK 8, we do need a long-term fix for this Xcode version check. I propose we remove the version check from JDK 8 gcc.make file to bring its behavior in line with JDK 9 and later.
BTW, I have tested [1] and confirmed that we still need to disable optimization on both loopTransform.o and unsafe.o with toolchain as recent as Xcode 11.3.
[0] I've never actually found any evidence that these two issues are in fact the result of clang bugs as opposed to bugs in our own code. Perhaps something to investigate in the future.
[1]
jdk/test/sun/misc/CopyMemory.java
hotspot/test/compiler/c2/5091921/
BTW, I have tested [1] and confirmed that we still need to disable optimization on both loopTransform.o and unsafe.o with toolchain as recent as Xcode 11.3.
[0] I've never actually found any evidence that these two issues are in fact the result of clang bugs as opposed to bugs in our own code. Perhaps something to investigate in the future.
[1]
jdk/test/sun/misc/CopyMemory.java
hotspot/test/compiler/c2/5091921/
- backported by
-
JDK-8244874 remove clang version check for optimization bug workaround from 8u
- Resolved
-
JDK-8246868 remove clang version check for optimization bug workaround from 8u
- Resolved
-
JDK-8248934 remove clang version check for optimization bug workaround from 8u
- Resolved
-
JDK-8251612 remove clang version check for optimization bug workaround from 8u
- Resolved