On Wed, 25 May 2022 06:25:53 GMT, Jatin Bhateja <[email protected]> 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
