RFR: 8332920: C2: Partial Peeling is wrongly applied for CmpU with negative limit [v4]

Christian Hagedorn chagedorn at openjdk.org
Thu Jun 6 07:41:16 UTC 2024


On Thu, 6 Jun 2024 06:59:13 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Christian Hagedorn has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Add randomized tests
>
> src/hotspot/share/opto/loopopts.cpp line 3055:
> 
>> 3053: //  exit-region     |
>> 3054: //        |         |
>> 3055: //    dummy-if      |
> 
> Still, I don't see this `dummy-if` mentioned in any of the comments. Can you add a comment line about what this is, and where it comes from?

Fair point, added a note.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19522#discussion_r1628920348


More information about the hotspot-compiler-dev mailing list