[jdk21u-dev] RFR: 8310308: IR Framework: check for type and size of vector nodes [v3]

Matthias Baesken mbaesken at openjdk.org
Tue Dec 19 14:16:51 UTC 2023


On Mon, 18 Dec 2023 10:58:18 GMT, Goetz Lindenmaier <goetz at openjdk.org> wrote:

>> I backport this to improve testing of C2 in 22u. It will simplify backporting tests of fixes to C2, as, e.g. " 8313720: C2 SuperWord: wrong result with -XX:+UseVectorCmov -XX:+UseCMoveUnconditionally"
>> 
>> I had to trivially resolve three files:
>> 
>> test/hotspot/jtreg/compiler/loopopts/superword/TestDependencyOffsets.java
>> Resolved the bugid in comment.
>> 
>> patching file test/hotspot/jtreg/compiler/vectorization/runner/BasicDoubleOpTest.java
>> patching file test/hotspot/jtreg/compiler/vectorization/runner/BasicFloatOpTest.java
>> These had to be resolved because "8308340: C2: Idealize Fma nodes" is not in 21. Trivial.
>> That change is huge thus not a candidate for backport.
>> 
>> Further I include "8314513: [IR Framework] Some internal IR Framework tests are failing after JDK-8310308 on PPC and Cascade Lake".  This is a follow-up fix of the tests.
>> 
>> All tests passed locally. 
>> 
>> It's a test-only change.
>
> Goetz Lindenmaier has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Fix failing test - revert to status of 21

Marked as reviewed by mbaesken (Reviewer).

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

PR Review: https://git.openjdk.org/jdk21u-dev/pull/29#pullrequestreview-1788959445


More information about the jdk-updates-dev mailing list