

<HTML><BODY><div>Hi, Sergey!</div><div>Thanks for the patch!</div><div>LGTM</div><div data-signature-widget="container"><div data-signature-widget="content"><div>--<br>Best regards,</div><div>Maxim Kokryashkin</div></div></div><div> </div><div> </div><blockquote style="border-left:1px solid #0857A6; margin:10px; padding:0 0 0 10px;">Вторник, 7 ноября 2023, 14:15 +03:00 от Sergey Kaplun <skaplun@tarantool.org>:<br> <div id=""><div class="js-helper js-readmsg-msg"><div><div id="style_16993557321669469887_BODY">Hi, Igor!<br>Thanks for the review!<br>Fixed the nit and force-pushed the branch.<br><br>On 07.11.23, Igor Munkin wrote:<br>> Sergey,<br>><br>> Thanks for the patch! LGTM as obvious, except the single nit regarding<br>> the commit message.<br>><br>> On 07.11.23, Sergey Kaplun wrote:<br>> > From: Mike Pall <mike><br>> ><br>> > Reported by Sergey Kaplun.<br>> ><br>> > (cherry-picked from commit d133d67c881f363f0b5584ebd21a965eb3435aa1)<br>> ><br>> > This patch is the follow-up for the commit<br>> > 1672bdc0ffbee9f32fadb6943909d6af7eb9a7b1 ("x64: Fix __call metamethod<br>> > return dispatch."). This patch uses the incorrect macro x64 (which is<br>><br>> I don't get which one of the aforementioned patches is "this": *this*<br>> one fixing the issue, or *that* one, that doesn't.<br><br>Fixed to "That", as you suggested. Force-pushed the branch.<br><br>><br><br><snipped><br><br>> > diff --git a/src/vm_x86.dasc b/src/vm_x86.dasc<br>> > index 9fa9a3f7..6a353796 100644<br>> > --- a/src/vm_x86.dasc<br>> > +++ b/src/vm_x86.dasc<br>> > @@ -1451,7 +1451,7 @@ static void build_subroutines(BuildCtx *ctx)<br>> > | mov LFUNC:RB, [RA-8]<br>> > | add NARGS:RD, 1<br>> > | // This is fragile. L->base must not move, KBASE must always be defined.<br>> > - |.if x64<br>> > + |.if X64<br>><br>> Bonjour.<br><br>Aah! Et bonjour mes nouveaux (espérons-le) amis !<br><br>><br>> > | cmp KBASEa, rdx // Continue with CALLT if flag set.<br>> > |.else<br>> > | cmp KBASE, BASE // Continue with CALLT if flag set.<br>> > --<br>> > 2.42.0<br>> ><br>><br>> --<br>> Best regards,<br>> IM<br><br>--<br>Best regards,<br>Sergey Kaplun</div></div></div></div></blockquote><div> </div></BODY></HTML>