

<HTML><BODY><div>Hi, Sergey!</div><div>Thanks for the review!</div><div>Now this line looks like the following:</div><div><div><div>| The issue is solved by using `GCSize`, which has a size of</div><div>| 64 bits on GC64 builds, instead of `MSize`.</div></div></div><div> </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;">Понедельник, 5 февраля 2024, 12:57 +03:00 от Sergey Kaplun <skaplun@tarantool.org>:<br> <div id=""><div class="js-helper js-readmsg-msg"><div><div id="style_17071270731776014784_BODY">Hi, Maxim!<br>Thanks for the fixes!<br>LGTM, except a single nit below.<br><br>On 02.02.24, Maxim Kokryashkin wrote:<br>> Hi, Sergey!<br>> Thanks for the review!<br>> Fixed your comments, branch is force-pushed.<br>> Here is the diff with changes:<br><br><snipped><br><br>> New commit message:<br>> ===<br>> LJ_GC64: Fix HREFK optimization.<br>><br>> Contributed by XmiliaH.<br>><br>> (cherry-picked from commit 91bc6b8ad1f373c1ce9003dc024b2e21fad0e444)<br>><br>> In `lj_record_idx` when `ix->oldv` is the global nilnode and the<br>> required key is not present in the table, it is possible to pass<br>> the constant key lookup optimization condition because of the<br>> `uint32_t` (`MSize`) overflow. Because of that, further recording<br>> incorrectly removes the check for the nilnode, which produces<br>> wrong results when trace is called for a different table.<br>><br>> The issue is solved by using `GCSize`, which has a size of<br><br>Minor: I'd say 'which size is defines by the target system' or something<br>like that (to mention that it is 32 bit for non-GC64 build).<br><br>> 64 bits, instead of `MSize`.<br>><br>> Maxim Kokryashkin:<br>> * added the description and the test for the problem<br>><br>> Part of tarantool/tarantool#9145<br>> ===<br><br><snipped><br><br>> ><br>> > --<br>> > Best regards,<br>> > Sergey Kaplun<br><br>--<br>Best regards,<br>Sergey Kaplun</div></div></div></div></blockquote><div> </div></BODY></HTML>