Integrated: 8323577: C2 SuperWord: remove AlignVector restrictions on IR tests added in JDK-8305055

Emanuel Peter epeter at openjdk.org
Mon Jan 15 10:43:29 UTC 2024


On Thu, 11 Jan 2024 10:16:14 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

> These IR rules were restricted in [JDK-8305055](https://bugs.openjdk.org/browse/JDK-8305055), [PR for comparison](https://git.openjdk.org/jdk/pull/13236).
> 
> This had to be done because those cases were no longer vectorized with `AlignVector` after my bugfix [JDK-8298935](https://bugs.openjdk.org/browse/JDK-8298935), they were "collateral damage". Before this bugfix, we would vectorize, even though the alignment constraints had rejected some memops, but then they were re-intriduced during pair extension. This re-introduction led to this bug for other reasons. I proposed to restore vectorization (i.e. fix the "collateral damage") by improving alignment-constraints ([JDK-8303827](https://bugs.openjdk.org/browse/JDK-8303827)).
> 
> Since I had to already completely rework the alignment constraints because of a bug, I relaxed the constraints:
> [JDK-8310190](https://bugs.openjdk.org/browse/JDK-8310190)
> 
> Now I can remove the restrictions on those rules.

This pull request has now been integrated.

Changeset: 45c65e6b
Author:    Emanuel Peter <epeter at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/45c65e6b1ac06aa06757393f1752661252e6f827
Stats:     13 lines in 2 files changed: 0 ins; 12 del; 1 mod

8323577: C2 SuperWord: remove AlignVector restrictions on IR tests added in JDK-8305055

Reviewed-by: rcastanedalo, chagedorn

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

PR: https://git.openjdk.org/jdk/pull/17369


More information about the hotspot-compiler-dev mailing list