RFR: 8323582: C2 SuperWord AlignVector: misaligned vector memory access with unaligned native memory [v3]

[Roland Westrelin]

On Tue, 18 Feb 2025 09:42:17 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> src/hotspot/share/opto/loopUnswitch.cpp line 513:
>> 
>>> 511: 
>>> 512:   // Create new Region.
>>> 513:   RegionNode* region = new RegionNode(1);
>> 
>> So we create a new `Region` every time a new condition is added?
>
> Yes. Are you ok with that? Or would you prefer if we extended an existing region (is that possible?) and then we'd have 2 cases, one where there is none yet, and one where we'd extend. I think adding one each time is easier, and it would get commoned anyway, right?

That sounds ok to me.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22016#discussion_r1963217281