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