Integrated: 8338124: C2 SuperWord: MulAddS2I input permutation still partially broken after JDK-8333840
Emanuel Peter
epeter at openjdk.org
Tue Aug 13 05:56:04 UTC 2024
On Mon, 12 Aug 2024 07:17:22 GMT, Emanuel Peter <epeter at openjdk.org> wrote:
> [JDK-8333840](https://bugs.openjdk.org/browse/JDK-8333840) attempted to fix some input permutation cases. Sadly, there is a typo which means that there can still be some broken cases that produce wrong results.
>
> `MulAddS2I` is a special case, where we have to verify that the 4 inputs for all members of the pack are in the same permutation. I only started the checking on the second element. This worked on all my previous examples, but does not work on the added regression test. We must, of course, start the checking on the first element.
This pull request has now been integrated.
Changeset: c27a8c8c
Author: Emanuel Peter <epeter at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/c27a8c8c8b867e6812b905f6154762802a498dbd
Stats: 18 lines in 2 files changed: 16 ins; 0 del; 2 mod
8338124: C2 SuperWord: MulAddS2I input permutation still partially broken after JDK-8333840
Reviewed-by: chagedorn, thartmann, kvn
-------------
PR: https://git.openjdk.org/jdk/pull/20539
More information about the hotspot-compiler-dev
mailing list