[jdk17] RFR: 8269568: JVM crashes when running VectorMask query tests [v2]

Xiaohong Gong xgong at openjdk.java.net
Fri Jul 2 04:30:29 UTC 2021


On Thu, 1 Jul 2021 15:04:04 GMT, Paul Sandoz <psandoz at openjdk.org> wrote:

>> Xiaohong Gong has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Refactor VectorMask query tests
>>  - Fix x86 backend codegen issue
>
> test/jdk/jdk/incubator/vector/templates/Unit-Miscellaneous.template line 425:
> 
>> 423:     static void maskTrueCount$vectorteststype$SmokeTest(IntFunction<boolean[]> fa) {
>> 424:         boolean[] a = fa.apply(SPECIES.length());
>> 425: #if[Int]
> 
> We already know the length of `r`, it's `a.length`, so we can just do:
> ```java 
>         int[] r = new int[a.length];
> 
> Thereby reducing the complexity. I realize you are copying the pattern of `lfr` which i think is also unnecessary. Let's clean this up later.

Resolved. Thanks for the suggestion!

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

PR: https://git.openjdk.java.net/jdk17/pull/168


More information about the hotspot-compiler-dev mailing list