[jdk21u-dev] Integrated: 8316594: C2 SuperWord: wrong result with hand unrolled loops

Goetz Lindenmaier goetz at openjdk.org
Mon Jan 8 09:16:28 UTC 2024


On Tue, 2 Jan 2024 17:16:18 GMT, Goetz Lindenmaier <goetz at openjdk.org> wrote:

> I backport this for parity with 21.0.3-oracle.
> 
> I had to resove SWPointer / VPointer because 8312332: C2: Refactor SWPointer out from SuperWord is not in 21.
> 
> We had previous backports that needed to be resolved because of this refactoring missing. I assume this will hit us in future repeatedly.
> 
> Thus I had a look at backporting 8312332, but that depends on three other cleanup changes:
> 8305636: Expand and clean up predicate classes and move them into separate files
> 8311691: C2: Remove legacy code related to PostLoopMultiversioning
> 8308606: C2 SuperWord: remove alignment checks when not required
> 
> All four of them can be backported clean, but 8305363 has four follow-up bugs.
> 
> So maybe we need to go the path and accept the unclean backports right from the beginning of 21u :(

This pull request has now been integrated.

Changeset: db1fa92d
Author:    Goetz Lindenmaier <goetz at openjdk.org>
URL:       https://git.openjdk.org/jdk21u-dev/commit/db1fa92d7420dc94f82af0c781a04a6e0ca511eb
Stats:     112 lines in 3 files changed: 104 ins; 5 del; 3 mod

8316594: C2 SuperWord: wrong result with hand unrolled loops

Reviewed-by: roland, shade
Backport-of: 3630af26e6f05237195e53b98603adf404e90a1a

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

PR: https://git.openjdk.org/jdk21u-dev/pull/110


More information about the jdk-updates-dev mailing list