On Thu, 10 Feb 2022 18:55:29 GMT, Paul Sandoz <[email protected]> wrote:
>> Quan Anh Mai has updated the pull request incrementally with two additional
>> commits since the last revision:
>>
>> - minor rename
>> - address reviews
>
> 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.
-------------
PR: https://git.openjdk.java.net/jdk/pull/7358
