RFR: 8284960: Integration of JEP 426: Vector API (Fourth Incubator) [v8]

Jatin Bhateja jbhateja at openjdk.java.net
Thu May 26 06:22:09 UTC 2022


On Wed, 25 May 2022 06:25:53 GMT, Jatin Bhateja <jbhateja at openjdk.org> wrote:

>> src/hotspot/cpu/x86/assembler_x86.cpp line 8173:
>> 
>>> 8171: 
>>> 8172: void Assembler::vinsertf32x4(XMMRegister dst, XMMRegister nds, XMMRegister src, uint8_t imm8) {
>>> 8173:   assert(VM_Version::supports_evex(), "");
>> 
>> Hmm, did we never trigger this wrong assert because the use was guarded by correct check?
>
> Yes.

> @jatin-bhateja something wrong with merge. `vpadd()` is removed. It was added by #8778 and still is used in `x86.ad`.

Hi @vnkozlov , after integration of PR 8778 there were there were two copies of vpadd with same signature, so removed one of them.

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

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


More information about the hotspot-compiler-dev mailing list