RFR: 8310190: C2 SuperWord: AlignVector is broken, generates misaligned packs [v21]

[Christian Hagedorn]

On Mon, 4 Dec 2023 13:30:28 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Emanuel Peter has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Suggestions by Christian for naming
>
> src/hotspot/share/opto/superword.cpp line 1720:
> 
>> 1718:   //        + offset              + offset                                  + C_const                   (sum of constant terms)
>> 1719:   //        + invar               + invar_factor * var_invar                + C_invar * var_invar       (term for variable init)
>> 1720:   //                          /   + scale * init                            + C_init  * var_init        (term for invariant)
> 
> You flipped the comments for the variable init and invariant terms

`var_invar` is slightly confusing. Maybe we should flip it to `invar_var` to be consistent with `invar_factor`? Or maybe you find another name for that term. I could not come up with something better for now.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14785#discussion_r1413876208