On Sun, 13 Feb 2022 05:14:47 GMT, Quan Anh Mai <[email protected]> wrote:
>> Observing the following failures on CPUs with
>> "Intel_R__Xeon_R__Gold_6354_CPU___3.00GHz" with HotSpot flags:
>>
>> -XX:+CreateCoredumpOnCrash -ea -esa -XX:CompileThreshold=100
>> -XX:+UnlockExperimentalVMOptions -server -XX:-TieredCompilation
>>
>>
>> TestVectorCastAVX512.java:
>>
>> Failed IR Rules (1)
>> ------------------
>> - Method "public static void
>> compiler.vectorapi.reshape.tests.TestVectorCast.testUI256toL512(int[],long[])":
>> * @IR rule 1: "@compiler.lib.ir_framework.IR(failOn={}, applyIf={},
>> applyIfAnd={}, applyIfOr={},
>> counts={"(\\\\d+(\\\\s){2}(VectorUCastI2X.*)+(\\\\s){2}===.*)", "1"},
>> applyIfNot={})"
>> - counts: Graph contains wrong number of nodes:
>> Regex 1: (\\d+(\\s){2}(VectorUCastI2X.*)+(\\s){2}===.*)
>> Expected 1 but found 0 nodes.
>>
>>
>> TestVectorCastAVX1.java:
>>
>> - Method "public static void
>> compiler.vectorapi.reshape.tests.TestVectorCast.testUB64toS64(byte[],short[])":
>> * @IR rule 1: "@compiler.lib.ir_framework.IR(failOn={}, applyIf={},
>> applyIfAnd={}, applyIfOr={},
>> counts={"(\\\\d+(\\\\s){2}(VectorUCastB2X.*)+(\\\\s){2}===.*)", "1"},
>> applyIfNot={})"
>> - counts: Graph contains wrong number of nodes:
>> Regex 1: (\\d+(\\s){2}(VectorUCastB2X.*)+(\\s){2}===.*)
>> Expected 1 but found 0 nodes.
>>
>> - Method "public static void
>> compiler.vectorapi.reshape.tests.TestVectorCast.testUB64toI128(byte[],int[])":
>> * @IR rule 1: "@compiler.lib.ir_framework.IR(failOn={}, applyIf={},
>> applyIfAnd={}, applyIfOr={},
>> counts={"(\\\\d+(\\\\s){2}(VectorUCastB2X.*)+(\\\\s){2}===.*)", "1"},
>> applyIfNot={})"
>> - counts: Graph contains wrong number of nodes:
>> Regex 1: (\\d+(\\s){2}(VectorUCastB2X.*)+(\\s){2}===.*)
>> Expected 1 but found 0 nodes.
>
> @PaulSandoz Thanks a lot for your testing, the reason seems to be due to
> `LaneType::asIntegral` missing `ForceInline` annotation. I have run the
> reshape test 10 times without getting any failure while with previous patch
> there is often 1 or 2.
> Thanks.
@merykitty testing now passes. Java bits look good. Needs HotSpot reviewer.
-------------
PR: https://git.openjdk.java.net/jdk/pull/7358
