RFR: 8319451: PhaseIdealLoop::conditional_move is too conservative [v2]

Thu Dec 14 15:04:49 UTC 2023

On Mon, 13 Nov 2023 07:49:25 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> Quan Anh Mai 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 three additional commits since the last revision:
>> 
>>  - change freq to double
>>  - Merge branch 'master' into cmovethreshold
>>  - adjust threshold, add benchmark
>
> Looks reasonable to me. All tests passed and performance results look neutral (no statistically significant improvements or regressions).

@TobiHartmann @vnkozlov @cl4es Thanks a lot for your reviews and testings, if there is nothing that concerns you, I will integrate the patch,

-------------

PR Comment: https://git.openjdk.org/jdk/pull/16524#issuecomment-1856012645