On Wed, 25 May 2022 06:25:53 GMT, Jatin Bhateja <jbhat...@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

Reply via email to