[lworld] RFR: 8241910: [lworld] C2 crashes due to duplicate storestore barrier added by JDK-8236522

Tobias Hartmann thartmann at openjdk.java.net
Tue Mar 31 08:42:21 UTC 2020


The fix for JDK-8236522 contained the partial/broken fix for a missing barrier issue that was already fixed by
JDK-8238780. As a result, tests are failing:

compiler/valhalla/valuetypes/TestIntrinsics.java macosx-x64-debug Exception: java.lang.RuntimeException: Expected to
get exit value of [0] compiler/valhalla/valuetypes/TestIntrinsics.java macosx-x64-debug Exception:
java.lang.RuntimeException: Expected to get exit value of [0] compiler/valhalla/valuetypes/TestBufferTearing.java
macosx-x64-debug ExitCode: 134 compiler/valhalla/valuetypes/TestBufferTearing.java macosx-x64-debug ExitCode: 134
compiler/valhalla/valuetypes/TestIntrinsics.java linux-x64-debug Exception: java.lang.RuntimeException: Expected to get
exit value of [0] compiler/valhalla/valuetypes/TestBufferTearing.java linux-x64-debug ExitCode: 134
compiler/valhalla/valuetypes/TestIntrinsics.java linux-x64-open-debug Exception: java.lang.RuntimeException: Expected
to get exit value of [0] compiler/valhalla/valuetypes/TestBufferTearing.java linux-x64-open-debug ExitCode: 134

The changes to valuetypenode.cpp/hpp should be reverted.

-------------

Commit messages:
 - 8241910: [lworld] C2 crashes due to duplicate storestore barrier added by JDK-8236522

Changes: https://git.openjdk.java.net/valhalla/pull/8/files
 Webrev: https://webrevs.openjdk.java.net/valhalla/8/webrev.00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8241910
  Stats: 18 lines in 2 files changed: 0 ins; 16 del; 2 mod
  Patch: https://git.openjdk.java.net/valhalla/pull/8.diff
  Fetch: git fetch https://git.openjdk.java.net/valhalla pull/8/head:pull/8

PR: https://git.openjdk.java.net/valhalla/pull/8


More information about the valhalla-dev mailing list