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