RFR: 8324751: C2 SuperWord: Aliasing Analysis runtime check [v11]
    Vladimir Kozlov 
    kvn at openjdk.org
       
    Sun Aug 17 22:55:26 UTC 2025
    
    
  
On Sun, 17 Aug 2025 21:42:01 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:
>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   more documentation for Christian
>
> src/hotspot/share/opto/mempointer.hpp line 403:
> 
>> 401: //   Given:
>> 402: //     (C0) pointer p and its MemPointer mp, which is constructed with safe decompositions.
>> 403: //     (C1) a summand "scale_v * v" that occurs in mp.
> 
> What is `v` here? And related `scale_v`?  In previous text you used `scale_i * variable_i`. Is it the same.
Based on new comment at line 44 this is induction variable (iv). Consider explaining that `v` is induction variable to avoid confusion.  I first thought it represents all kind of variables used in index expressions.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24278#discussion_r2281033664
    
    
More information about the hotspot-compiler-dev
mailing list