On Wed, 5 Mar 2025 11:39:05 GMT, Jatin Bhateja <jbhat...@openjdk.org> wrote:
>> Ferenc Rakoczi has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains three commits: >> >> - Merged master. >> - Added comments, removed debugging printfs >> - JDK-8351034 Add AVX-512 intrinsics for ML-DSA > > src/hotspot/cpu/x86/stubGenerator_x86_64_dilithium.cpp line 292: > >> 290: __ movl(iterations, 2); >> 291: >> 292: __ BIND(L_loop); > > Hi @ferakocz , Kindly align loop entry address using __align64() here and at > all the places before __BIND(LOOP) Hi, @jatin-bhateja, thanks for the suggestion. I have added __ align(OptoLoopAlignment); before all loop entries. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/23860#discussion_r1981364481
