On Tue, 25 Oct 2022 23:48:49 GMT, Sandhya Viswanathan <sviswanat...@openjdk.org> wrote:
>> vpaprotsk has updated the pull request incrementally with one additional >> commit since the last revision: >> >> extra whitespace character > > src/hotspot/cpu/x86/macroAssembler_x86_poly.cpp line 806: > >> 804: evmovdquq(A0, Address(rsp, 64*0), Assembler::AVX_512bit); >> 805: evmovdquq(A0, Address(rsp, 64*1), Assembler::AVX_512bit); >> 806: evmovdquq(A0, Address(rsp, 64*2), Assembler::AVX_512bit); > > This is load from stack into A0. Did you intend to store A0 (cleanup) into > stack local area here? I think the source and destination are mixed up here. Wow! Thank you for spotting this ------------- PR: https://git.openjdk.org/jdk/pull/10582
