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

Christian Hagedorn chagedorn at openjdk.org
Thu Dec 7 07:35:45 UTC 2023


On Thu, 7 Dec 2023 07:27:43 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Thanks. Okay, I see your point. I agree that we can go with `aw` in the local scope of `pack_alignment_solution`/`AlignmentSolver`  for readability purposes (and you explain what it is in the comments/print statements). However, I think the `AlignmentSolution` class should use the full name `alignment_width` as a general property to store/return. What are your thoughts about that?
>
> Ok, that sounds good! I did the update.

Great, thanks! :-)

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

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


More information about the hotspot-compiler-dev mailing list