RFR: 8347273: C2: VerifyIterativeGVN for Ideal and Identity [v7]

Emanuel Peter epeter at openjdk.org
Thu Jun 12 07:54:02 UTC 2025


On Thu, 12 Jun 2025 06:50:25 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   reorder flags for Christian
>
> Some more minor comments but otherwise looks good, thanks for the updates and for the offline discussion to share some background!
> 
>> @chhagedorn I checked with @TobiHartmann : he said he does not have a strong opinion, but if he had to make a decision, he would prefers having everything in the comments.
> 
> Then let's go with everything in the comments - might be better in the end since we eventually want to replace the individual comments with final verdicts or have the cases fixed anyway at some point :-)

@chhagedorn Thanks for reviewing and the suggestions! I addressed them all :)

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

PR Comment: https://git.openjdk.org/jdk/pull/22970#issuecomment-2965521510


More information about the hotspot-dev mailing list