RFR: 8323079: Regression of -5% to -11% with SPECjvm2008-MonteCarlo after JDK-8319451 [v2]
Quan Anh Mai
qamai at openjdk.org
Tue Jun 11 17:35:32 UTC 2024
> Hi,
>
> I cannot explain the regression, comparing the current mainline to JDK-21 reveals a decrease in performance, yet it is only for some combinations of OS-GC and perfasm shows that the hot regions (>99% of execution time) do not contain differences that can explain the results.
>
> Consequently, with the advice of @TobiHartmann , I propose to effectively revert JDK-8319451 for the generation of `CMove`s inside loops. For those outside, the before-JDK-8319451 probability threshold is 0.001 and the current value is 0.01. I think the current value is more reasonable as evidenced by the benchmark added in JDK-8319451.
>
> Please kindly review, thank you very much.
> Quan Anh
Quan Anh Mai has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:
Revert "8319451: PhaseIdealLoop::conditional_move is too conservative"
This reverts commit ac968c36d7cc2e13270d28c9310178f6b654d7dc.
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/19650/files
- new: https://git.openjdk.org/jdk/pull/19650/files/0c0c9780..caaee62b
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=19650&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=19650&range=00-01
Stats: 74 lines in 2 files changed: 9 ins; 61 del; 4 mod
Patch: https://git.openjdk.org/jdk/pull/19650.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/19650/head:pull/19650
PR: https://git.openjdk.org/jdk/pull/19650
More information about the hotspot-compiler-dev
mailing list