RFR: 8335392: C2 MergeStores: enhanced pointer parsing [v13]

[Emanuel Peter]

On Fri, 1 Nov 2024 13:52:48 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> src/hotspot/share/opto/mempointer.hpp line 400:
>> 
>>> 398: 
>>> 399:   // Use case: exact aliasing and adjacency.
>>> 400:   bool is_always_at_distance(const jint distance) const {
>> 
>> The "always" seems to refer to the `Always` but it reads like we are just curious about the distance. Is `is_always_and_at_distance()` more clear?
>
> Hmm. Maybe I can call it `is_always_with_distance`? Because this would imply that the two pointers always have an aliasing with this exact distance.... so that would be fitting in its **meaning**. But yours is more exactly what id **does**.... hmm.. what do you think?

Hmm. No I think I really like the original, because it reads like this:
`aliasing.is_always_at_distance(d)`.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19970#discussion_r1825905560