RFR: 8320237 C2: late inlining of method handle invoke causes duplicate lines in PrintInlining output [v3]
Tobias Hartmann
thartmann at openjdk.org
Mon Jan 29 10:07:46 UTC 2024
On Mon, 22 Jan 2024 16:28:02 GMT, Kangcheng Xu <kxu at openjdk.org> wrote:
>> 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
Okay, I'll re-run testing and report back once it passed.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/17147#issuecomment-1914357522
More information about the hotspot-compiler-dev
mailing list