RFR: 8343685: C2 SuperWord: refactor VPointer with MemPointer [v13]

[Emanuel Peter]

On Thu, 16 Jan 2025 15:02:39 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   refactor to has_invar_summands and print invar and fix non-product guards
>
> src/hotspot/share/opto/vectorization.hpp line 723:
> 
>> 721: //       corresponding scale. If there is no such summand, then we know that the
>> 722: //       pointer does not depend on the iv, since otherwise there would have to be
>> 723: //       a summand where its variable is main-loop variant.
> 
> Could we find multiple summands with iv?

No, they would have all been collected into a single summand by `MemPointer` - that is one of the powers of `MemPointer`. Do you want me to leave a comment here for that?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21926#discussion_r1918804329