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-compiler-dev
mailing list