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