RFR: 8310190: C2 SuperWord: AlignVector is broken, generates misaligned packs [v22]
Emanuel Peter
epeter at openjdk.org
Thu Dec 7 15:15:14 UTC 2023
On Wed, 6 Dec 2023 12:36:30 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
>
> Impressive work! I'm still working my way through the proofs but here are some first comments.
@chhagedorn @fg1417 Ok, I now moved basically all code over to the `AlignmentSolver`. I placed it in `vectorization.hpp/cpp`, since I plan to use this facility in the future, possibly outside of SuperWord.
I'd be very thankful for re-reviews ;)
-------------
PR Comment: https://git.openjdk.org/jdk/pull/14785#issuecomment-1845523575
More information about the hotspot-compiler-dev
mailing list