RFR: 8316130: Incorrect control in LibraryCallKit::inline_native_notify_jvmti_funcs [v2]
Tobias Hartmann
thartmann at openjdk.org
Thu Sep 14 12:17:20 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
Tobias Hartmann has updated the pull request incrementally with one additional commit since the last revision:
Removed second vt_oop declaration
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/15713/files
- new: https://git.openjdk.org/jdk/pull/15713/files/88cc6dd4..7f49aa01
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=15713&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=15713&range=00-01
Stats: 1 line in 1 file changed: 0 ins; 1 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