<!DOCTYPE html>
<html data-lt-installed="true">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body style="padding-bottom: 1px;">
<p>Hi, Sergey,</p>
<p>LGTM patch series.</p>
<div class="moz-cite-prefix">On 3/2/26 11:05, Sergey Kaplun wrote:<br>
</div>
<blockquote type="cite"
cite="mid:cover.1772438261.git.skaplun@tarantool.org">
<pre wrap="" class="moz-quote-pre">This patch-set fixes the allocation limit for the build with disabled
JIT. The first commit helps to catch the error early in tests by the
correct check of the address returned by the allocator.
Branch: <a class="moz-txt-link-freetext" href="https://github.com/tarantool/luajit/tree/skaplun/lj-1430-alloc-limit">https://github.com/tarantool/luajit/tree/skaplun/lj-1430-alloc-limit</a>
Related issues:
* <a class="moz-txt-link-freetext" href="https://github.com/LuaJIT/LuaJIT/issues/1430">https://github.com/LuaJIT/LuaJIT/issues/1430</a>
* <a class="moz-txt-link-freetext" href="https://github.com/tarantool/tarantool/issues/12134">https://github.com/tarantool/tarantool/issues/12134</a>
Related ML thread: <a class="moz-txt-link-freetext" href="https://www.freelists.org/post/luajit/Provide-error-outputdetails-upon-luaL-newstate-failures">https://www.freelists.org/post/luajit/Provide-error-outputdetails-upon-luaL-newstate-failures</a>
Mike Pall (2):
Fix pointer check for non-GC64 mode.
x64/!LJ_GC64: The allocation limit is required for a no-JIT build,
too.
src/lj_alloc.c | 4 +-
src/lj_def.h | 3 +-
.../lj-1430-internal-alloc-limit.test.lua | 39 +++++++++++++++++++
3 files changed, 43 insertions(+), 3 deletions(-)
create mode 100644 test/tarantool-tests/lj-1430-internal-alloc-limit.test.lua
</pre>
</blockquote>
</body>
<lt-container></lt-container>
</html>