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

Emanuel Peter epeter at openjdk.org
Wed Dec 6 17:04:08 UTC 2023


On Wed, 6 Dec 2023 12:00:21 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Emanuel Peter has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - add newline suggested by Faye
>>  - improve the alignment proof, make it more explicit
>
> src/hotspot/share/opto/superword.cpp line 1894:
> 
>> 1892:   //   for any pre_iter_C_const >= 0: C_pre * pre_iter_C_const = 0 (mod aw)
>> 1893:   //
>> 1894:   // which implies that C_iter (and pre_iter_C_const) have no effect on the alignment of
> 
> What is `C_iter`?

typo. `pre_iter`

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

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


More information about the hotspot-compiler-dev mailing list