RFR: 8331764: C2 SuperWord: refactor _align_to_ref/_mem_ref_for_main_loop_alignment

Wed May 8 04:46:53 UTC 2024

On Wed, 8 May 2024 04:43:46 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Yes, it is. All of this code is run in `SuperWord::output`, and at this point we are committed to vectorization - everything is verified.
>
> That is what I tried to say in the PR description:
>> It makes more sense to pick a mem_ref directly in SuperWord::adjust_pre_loop_limit_to_align_main_loop_vectors, where we already know what packs are going to be vectorized.

Yes, `first->memory_size()` knows the size in bytes of the load/store. It is used many places in SuperWord.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19115#discussion_r1593377294