RFR: 8334629: [BACKOUT] PhaseIdealLoop::conditional_move is too conservative [v2]

[Quan Anh Mai]

On Thu, 13 Jun 2024 05:42:03 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> 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.
>
> The OpenJDK Guide has all the details, see "Alternative 3":
> https://openjdk.org/guide/#backing-out-a-change
> 
> I think creating a REDO is fine though, since we definitely want to get rid of the dependency on `BlockLayoutMinDiamondPercentage` at some point. Feel free to leave it unassigned though.

@TobiHartmann Thanks for your suggestion, I have made relevant issues and changed the title of this PR, can you please review it?

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

PR Comment: https://git.openjdk.org/jdk/pull/19650#issuecomment-2180587012