Integrated: 8296638: RISC-V: NegVI node emits wrong code when vector element basic type is T_BYTE/T_SHORT

[Dingli Zhang]

On Thu, 10 Nov 2022 01:43:40 GMT, Dingli Zhang <dzhang at openjdk.org> wrote:

> Hi,
> 
> test/jdk/jdk/incubator/vector/Byte256VectorTests.java fails on riscv with the following error:
> 
> test Byte256VectorTests.negByte256VectorTests (byte [i * 5]): failure
> java.lang.AssertionError: at index #2, input = 10 expected [-10] but found [-11]
> 
> 
> Currently, `NegVI` can only handle the vector element basic type `T_INT` with`vsetvli(t0, x0, Assembler::e32)` but `T_SHORT/T_BYTE` can also be matched with `NegVI`, so these two types of tests are currently failing:
> 
> test/jdk/jdk/incubator/vector/Byte*VectorTests.java
> test/jdk/jdk/incubator/vector/Short*VectorTests.java
> 
> 
> Meanwhile, I removed some useless trailing whitespace.
> 
> Please take a look and have some reviews. Thanks a lot.
> 
> ## Testing:
> 
> - test/jdk/jdk/incubator/vector (fastdebug/release with UseRVV on QEMU)

This pull request has now been integrated.

Changeset: f2acdfdc
Author:    Dingli Zhang <dzhang at openjdk.org>
Committer: Aleksey Shipilev <shade at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/f2acdfdcbd2a49c1167656e73b67b38b545f9472
Stats:     9 lines in 1 file changed: 2 ins; 0 del; 7 mod

8296638: RISC-V: NegVI node emits wrong code when vector element basic type is T_BYTE/T_SHORT

Reviewed-by: gcao, fyang, shade

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

PR: https://git.openjdk.org/jdk/pull/11074