RFR: 8267663: [vector] Add unsigned comparison operators on AArch64 [v2]
Andrew Haley
aph at openjdk.java.net
Wed Jun 9 08:51:17 UTC 2021
On Tue, 8 Jun 2021 14:53:07 GMT, Eric Liu <eliu at openjdk.org> wrote:
>> src/hotspot/cpu/aarch64/macroAssembler_aarch64.cpp line 5373:
>>
>>> 5371: case BoolTest::gt: fcmgt(dst, size, src1, src2); break;
>>> 5372: case BoolTest::le: fcmge(dst, size, src2, src1); break;
>>> 5373: case BoolTest::lt: fcmgt(dst, size, src2, src1); break;
>>
>> Are you sure about these two lines?
>
> As A64-SIMD doesn't have `fcmlt(register)` and `fcmle(register)` instructions, I think it's okay to use `fcmgt` and `fcmge` with reversing the argument. But I can't locate the initial patch adding them.
Yes, I see now.
-------------
PR: https://git.openjdk.java.net/jdk/pull/4358
More information about the hotspot-compiler-dev
mailing list