RFR: 8316130: Incorrect control in LibraryCallKit::inline_native_notify_jvmti_funcs

Tobias Hartmann thartmann at openjdk.org
Wed Sep 13 11:58:04 UTC 2023


We hit an assert during loop opts because the control input fed into `must_be_not_null` in `LibraryCallKit::inline_native_notify_jvmti_funcs` is wrong. The problem is that control is obtained from the `GraphKit` while it's been updated via the `IdealKit`. I simply moved the `must_be_not_null` out of the if branch and to before `IdealKit` creation, similar to what we do for other intrinsics.

The original reproducer requires JFR and changes to core libraries (see JBS for details) and I was not able to extract a standalone reproducer. I don't think it's worth it because the required core libraries changes will be integrated separately and then the existing test will trigger the issue (with JFR).

Thanks,
Tobias

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

Commit messages:
 - 8316130: Incorrect control in LibraryCallKit::inline_native_notify_jvmti_funcs

Changes: https://git.openjdk.org/jdk/pull/15713/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=15713&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8316130
  Stats: 5 lines in 1 file changed: 2 ins; 3 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/15713.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/15713/head:pull/15713

PR: https://git.openjdk.org/jdk/pull/15713


More information about the hotspot-compiler-dev mailing list