RFR: 8348263: C2 SuperWord: TestMemorySegment.java has failing IR rules with AlignVector after JDK-8343685

Emanuel Peter epeter at openjdk.org
Wed Jan 22 12:05:43 UTC 2025


On Wed, 22 Jan 2025 09:21:36 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> In [JDK-8343685](https://bugs.openjdk.org/browse/JDK-8343685) / https://git.openjdk.org/jdk/pull/21926 I enabled some IR rules, but forgot to test with `-XX:+AlignVector`. Now those rules fail and need a corresponding `applyIf`.
>> 
>> Additionally, I now enabled 2 more IR rules that I forgot to enable in https://git.openjdk.org/jdk/pull/21926.
>> 
>> To ensure we automatically run this test with  `-XX:+AlignVector`, I added some additional runs. I already did a similar thing with other tests that have IR rules that are sensitive to `AlignVector`.
>
> Looks good to me.

Thanks @TobiHartmann @chhagedorn for the reviews!
As discussed offline, I'll integrate quickly to reduce noise for people with the failing test. Internal testing with and without AlignVector are passing.

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

PR Comment: https://git.openjdk.org/jdk/pull/23229#issuecomment-2607059936


More information about the hotspot-compiler-dev mailing list