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