RFR: 8265956: JVM crashes when matching LShiftVB Node [v3]

Wang Huang whuang at openjdk.java.net
Sat May 8 08:41:34 UTC 2021


On Sat, 8 May 2021 08:30:41 GMT, Wang Huang <whuang at openjdk.org> wrote:

>> test/hotspot/jtreg/compiler/vectorapi/TestVectorShuffleIotaByte.java line 63:
>> 
>>> 61:     static byte[] expected_256 = {1, 3, 5, 7, 9, 11, 13, 15, 17, 19, 21, 23, 25, 27, 29, 31,
>>> 62:                                   -31, -29, -27, -25, -23, -21, -19, -17, -15, -13, -11, -9, -7, -5, -3, -1};
>>> 63:     static byte[] expected_512 = {1, 3, 5, 7, 9, 11, 13, 15, 17, 19, 21, 23, 25, 27, 29, 31,
>> 
>> Looks much better to me! Just a small suggestion here: it's better to use `static final` if these arrays are not expected to be modified.
>
> OK. Thank you for your review.

I have changed this.

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

PR: https://git.openjdk.java.net/jdk/pull/3747


More information about the hotspot-compiler-dev mailing list