RFR: 8320237 C2: late inlining of method handle invoke causes duplicate lines in PrintInlining output [v3]
Kangcheng Xu
kxu at openjdk.org
Mon Jan 22 16:28:02 UTC 2024
> This PR resolves [JDK-8320237](https://bugs.openjdk.org/browse/JDK-8320237)
>
> The original behavior produces both a failure and success message upon late inlining which is confusing. The patch removes the failure message if inlining was successful. Huge thanks to @rwestrel for reporting and working out a solution.
>
> Unit test `test/hotspot/jtreg/compiler/inlining/TestDuplicatedLateInliningOutput.java` is added and passing.
Kangcheng Xu has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains seven additional commits since the last revision:
- add -XX:-BackgroundCompilation flag
- Merge branch 'master' into JDK-8320237
- fix VM crashes
- update test summary, requirements, and VM flags
- Merge branch 'master' into JDK-8320237
- make regex whitespace consistent
and to trigger GHA
- 8320237: C2: late inlining of method handle invoke causes duplicate lines in PrintInlining output
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/17147/files
- new: https://git.openjdk.org/jdk/pull/17147/files/94d78fa1..cf23f46a
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=17147&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=17147&range=01-02
Stats: 39112 lines in 1107 files changed: 23690 ins; 10535 del; 4887 mod
Patch: https://git.openjdk.org/jdk/pull/17147.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/17147/head:pull/17147
PR: https://git.openjdk.org/jdk/pull/17147
More information about the hotspot-compiler-dev
mailing list