Hi, Sergey,
LGTM patch series.
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: https://github.com/tarantool/luajit/tree/skaplun/lj-1430-alloc-limit
Related issues:
* https://github.com/LuaJIT/LuaJIT/issues/1430
* https://github.com/tarantool/tarantool/issues/12134
Related ML thread: https://www.freelists.org/post/luajit/Provide-error-outputdetails-upon-luaL-newstate-failures
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