RFR: 8334431: C2 SuperWord: fix performance regression due to store-to-load-forwarding failures [v3]

[Emanuel Peter]

On Tue, 19 Nov 2024 14:40:20 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   use constant method handle to make the benchmark smaller
>
> src/hotspot/share/opto/vtransform.cpp line 154:
> 
>> 152: // Helper-class for VTransformGraph::has_store_to_load_forwarding_failure.
>> 153: // It represents a memory region: [ptr, ptr + memory_size)
>> 154: class VPointerRecord : public StackObj {
> 
> Not so sure about the name of this class. When first reading the code below (which I reviewed first), I found it difficult to understand its purpose without reading this class comment. How about `VMemoryRegion` or something like that to better show that it's about regions of memory and not single pointers?

@chhagedorn all `VPointer` are in fact memory-regions, and not just zero-length pointers. They have a `MemNode` which gives them a size. That is how we can compute the overlapping / aliasing queries with it...

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21521#discussion_r1848522853