Re: [Tarantool-patches] [PATCH v2 luajit 00/45] Rearrange LuaJIT misc tests

[Sergey Bronnikov via Tarantool-patches <tarantool-patches@dev.tarantool.org>]

[-- Attachment #1: Type: text/plain, Size: 10151 bytes --]

Sergey,

On 21.08.2024 11:58, Sergey Kaplun wrote:
> This patch set handles all tests from the <misc/> directory: removes
> duplicates or enables them as a part of other categories.
>
> Changes in v2:
> * Separate refactoring of the enabled tests, where the original
>    patch does not include the whole test due to its enabling and fixing
>    the indentation (as discussed with Sergey Bronnikov).
> * Updated the <LuaJIT-tests/README.md> to make it up-to-date.
>
> Branch:https://github.com/tarantool/luajit/tree/skaplun/gh-9398-more-luajit-tests-p2
> Issue:https://github.com/tarantool/tarantool/issues/9398
Thanks for patches! LGTM
> Sergey Kaplun (45):
>    test: don't run JIT-based LuaJIT tests without JIT
>    test: actualize <LuaJIT-tests/README.md>
>    test: enable <misc/alias_alloc.lua> LuaJIT test
>    test: refactor <alias_alloc.lua> LuaJIT test
>    test: refactor <lang/coroutine.lua> LuaJIT test
>    test: remove <misc/coro_yield.lua> LuaJIT test
>    test: enable <misc/debug_gc.lua> LuaJIT test
>    test: enable <misc/dualnum.lua> LuaJIT test
>    test: refactor <lang/dualnum.lua> LuaJIT test
>    test: remove <misc/fori_coerce.lua> LuaJIT test
>    test: remove <misc/fori_dir.lua> LuaJIT test
>    test: remove <misc/gc_rechain.lua> LuaJIT test
>    test: enable <misc/gc_trace.lua> LuaJIT test
>    test: refactor <trace/gc.lua> LuaJIT test
>    test: enable <misc/gcstep.lua> LuaJIT test
>    test: enable <misc/hook_active.lua> LuaJIT test
>    test: enable <misc/hook_line.lua> LuaJIT test
>    test: enable <misc/hook_norecord.lua> LuaJIT test
>    test: enable <misc/hook_record.lua> LuaJIT test
>    test: enable <misc/hook_top.lua> LuaJIT test
>    test: enable <misc/jit_flush.lua> LuaJIT test
>    test: remove <misc/loop_unroll.lua> LuaJIT test
>    test: enable <misc/parse_comp.lua> LuaJIT test
>    test: enable <misc/parse_esc.lua> LuaJIT test
>    test: enable <misc/parse_misc.lua> LuaJIT test
>    test: enable <misc/phi_conv.lua> LuaJIT test
>    test: refactor <trace/phi/conv.lua> LuaJIT test
>    test: enable <misc/recurse_deep.lua> LuaJIT test
>    test: remove <misc/recurse_tail.lua> LuaJIT test
>    test: enable <misc/stack_gc.lua> LuaJIT test
>    test: refactor <lang/gc_stack.lua> LuaJIT test
>    test: enable <misc/stack_purge.lua> LuaJIT test
>    test: refactor <trace/stack_purge.lua> LuaJIT test
>    test: enable <misc/stackov.lua> LuaJIT test
>    test: enable <misc/stackovc.lua> LuaJIT test
>    test: enable <misc/tcall_base.lua> LuaJIT test
>    test: refactor <trace/tcall_base.lua> LuaJIT test
>    test: enable <misc/tcall_loop.lua> LuaJIT test
>    test: enable <misc/tonumber_scan.lua> LuaJIT test
>    test: remove <misc/uclo.lua> LuaJIT test
>    test: enable <misc/unordered_jit.lua> LuaJIT test
>    test: enable <misc/wbarrier.lua> LuaJIT test
>    test: enable <misc/wbarrier_jit.lua> LuaJIT test
>    test: enable <misc/wbarrier_obar.lua> LuaJIT test
>    test: update <LuaJIT-tests/README.md>
>
>   test/LuaJIT-tests/CMakeLists.txt              |  10 +-
>   test/LuaJIT-tests/README.md                   |  17 +-
>   test/LuaJIT-tests/lang/coroutine.lua          |   8 -
>   .../deep_recursion.lua}                       |   8 +-
>   test/LuaJIT-tests/{misc => lang}/dualnum.lua  |  24 +-
>   .../{misc/debug_gc.lua => lang/gc_debug.lua}  |  31 ++-
>   .../{misc/stack_gc.lua => lang/gc_stack.lua}  |   6 +-
>   test/LuaJIT-tests/lang/gc_step.lua            |  39 +++
>   test/LuaJIT-tests/lang/hook_active.lua        | 102 ++++++++
>   test/LuaJIT-tests/lang/hook_line.lua          |  45 ++++
>   test/LuaJIT-tests/lang/hook_top.lua           |  59 +++++
>   test/LuaJIT-tests/lang/index                  |  15 +-
>   test/LuaJIT-tests/lang/parse_comp.lua         |  12 +
>   test/LuaJIT-tests/lang/parse_esc.lua          |  10 +
>   test/LuaJIT-tests/lang/parse_misc.lua         |  31 +++
>   test/LuaJIT-tests/lang/stackov.lua            |  48 ++++
>   test/LuaJIT-tests/lang/stackov_c.lua          |  12 +
>   test/LuaJIT-tests/lang/wbarrier.lua           |  11 +
>   test/LuaJIT-tests/lib/base/index              |   1 +
>   .../{misc => lib/base}/tonumber_scan.lua      |  58 +++--
>   test/LuaJIT-tests/lib/coroutine/index         |   1 +
>   test/LuaJIT-tests/lib/coroutine/traceback.lua |  14 +
>   test/LuaJIT-tests/misc/coro_traceback.lua     |   8 -
>   test/LuaJIT-tests/misc/coro_yield.lua         | 111 --------
>   test/LuaJIT-tests/misc/for_dir.lua            |  13 -
>   test/LuaJIT-tests/misc/fori_coerce.lua        |  33 ---
>   test/LuaJIT-tests/misc/gc_rechain.lua         |  32 ---
>   test/LuaJIT-tests/misc/gcstep.lua             |  33 ---
>   test/LuaJIT-tests/misc/hook_active.lua        |  95 -------
>   test/LuaJIT-tests/misc/hook_line.lua          |  41 ---
>   test/LuaJIT-tests/misc/hook_norecord.lua      |  12 -
>   test/LuaJIT-tests/misc/hook_record.lua        |   8 -
>   test/LuaJIT-tests/misc/hook_top.lua           |  55 ----
>   test/LuaJIT-tests/misc/jit_flush.lua          |  50 ----
>   test/LuaJIT-tests/misc/loop_unroll.lua        |  35 ---
>   test/LuaJIT-tests/misc/parse_comp.lua         |  13 -
>   test/LuaJIT-tests/misc/parse_esc.lua          |   7 -
>   test/LuaJIT-tests/misc/parse_misc.lua         |  31 ---
>   test/LuaJIT-tests/misc/recurse_tail.lua       |  22 --
>   test/LuaJIT-tests/misc/stackov.lua            |  40 ---
>   test/LuaJIT-tests/misc/stackovc.lua           |   4 -
>   test/LuaJIT-tests/misc/tcall_loop.lua         |   8 -
>   test/LuaJIT-tests/misc/uclo.lua               |  91 -------
>   test/LuaJIT-tests/misc/unordered_jit.lua      |  96 -------
>   test/LuaJIT-tests/misc/wbarrier.lua           |   7 -
>   test/LuaJIT-tests/misc/wbarrier_jit.lua       |  18 --
>   test/LuaJIT-tests/misc/wbarrier_obar.lua      |  22 --
>   test/LuaJIT-tests/opt/index                   |   1 +
>   .../{misc => opt/mem}/alias_alloc.lua         |  30 +--
>   test/LuaJIT-tests/opt/mem/index               |   1 +
>   .../{misc/gc_trace.lua => trace/gc.lua}       |  32 ++-
>   test/LuaJIT-tests/trace/hook_norecord.lua     |  12 +
>   test/LuaJIT-tests/trace/hook_record.lua       |   7 +
>   test/LuaJIT-tests/trace/index                 |   9 +
>   test/LuaJIT-tests/trace/jit_flush.lua         |  66 +++++
>   .../{misc/phi_conv.lua => trace/phi/conv.lua} |  37 +--
>   test/LuaJIT-tests/trace/phi/index             |   1 +
>   .../{misc => trace}/stack_purge.lua           |  14 +-
>   .../{misc => trace}/tcall_base.lua            |  14 +-
>   test/LuaJIT-tests/trace/tcall_loop.lua        |  10 +
>   test/LuaJIT-tests/trace/unordered_jit.lua     | 240 ++++++++++++++++++
>   test/LuaJIT-tests/trace/wbarrier.lua          |  57 +++++
>   62 files changed, 959 insertions(+), 1019 deletions(-)
>   delete mode 100644 test/LuaJIT-tests/lang/coroutine.lua
>   rename test/LuaJIT-tests/{misc/recurse_deep.lua => lang/deep_recursion.lua} (84%)
>   rename test/LuaJIT-tests/{misc => lang}/dualnum.lua (57%)
>   rename test/LuaJIT-tests/{misc/debug_gc.lua => lang/gc_debug.lua} (63%)
>   rename test/LuaJIT-tests/{misc/stack_gc.lua => lang/gc_stack.lua} (76%)
>   create mode 100644 test/LuaJIT-tests/lang/gc_step.lua
>   create mode 100644 test/LuaJIT-tests/lang/hook_active.lua
>   create mode 100644 test/LuaJIT-tests/lang/hook_line.lua
>   create mode 100644 test/LuaJIT-tests/lang/hook_top.lua
>   create mode 100644 test/LuaJIT-tests/lang/parse_comp.lua
>   create mode 100644 test/LuaJIT-tests/lang/parse_esc.lua
>   create mode 100644 test/LuaJIT-tests/lang/parse_misc.lua
>   create mode 100644 test/LuaJIT-tests/lang/stackov.lua
>   create mode 100644 test/LuaJIT-tests/lang/stackov_c.lua
>   create mode 100644 test/LuaJIT-tests/lang/wbarrier.lua
>   rename test/LuaJIT-tests/{misc => lib/base}/tonumber_scan.lua (93%)
>   create mode 100644 test/LuaJIT-tests/lib/coroutine/traceback.lua
>   delete mode 100644 test/LuaJIT-tests/misc/coro_traceback.lua
>   delete mode 100644 test/LuaJIT-tests/misc/coro_yield.lua
>   delete mode 100644 test/LuaJIT-tests/misc/for_dir.lua
>   delete mode 100644 test/LuaJIT-tests/misc/fori_coerce.lua
>   delete mode 100644 test/LuaJIT-tests/misc/gc_rechain.lua
>   delete mode 100644 test/LuaJIT-tests/misc/gcstep.lua
>   delete mode 100644 test/LuaJIT-tests/misc/hook_active.lua
>   delete mode 100644 test/LuaJIT-tests/misc/hook_line.lua
>   delete mode 100644 test/LuaJIT-tests/misc/hook_norecord.lua
>   delete mode 100644 test/LuaJIT-tests/misc/hook_record.lua
>   delete mode 100644 test/LuaJIT-tests/misc/hook_top.lua
>   delete mode 100644 test/LuaJIT-tests/misc/jit_flush.lua
>   delete mode 100644 test/LuaJIT-tests/misc/loop_unroll.lua
>   delete mode 100644 test/LuaJIT-tests/misc/parse_comp.lua
>   delete mode 100644 test/LuaJIT-tests/misc/parse_esc.lua
>   delete mode 100644 test/LuaJIT-tests/misc/parse_misc.lua
>   delete mode 100644 test/LuaJIT-tests/misc/recurse_tail.lua
>   delete mode 100644 test/LuaJIT-tests/misc/stackov.lua
>   delete mode 100644 test/LuaJIT-tests/misc/stackovc.lua
>   delete mode 100644 test/LuaJIT-tests/misc/tcall_loop.lua
>   delete mode 100644 test/LuaJIT-tests/misc/uclo.lua
>   delete mode 100644 test/LuaJIT-tests/misc/unordered_jit.lua
>   delete mode 100644 test/LuaJIT-tests/misc/wbarrier.lua
>   delete mode 100644 test/LuaJIT-tests/misc/wbarrier_jit.lua
>   delete mode 100644 test/LuaJIT-tests/misc/wbarrier_obar.lua
>   rename test/LuaJIT-tests/{misc => opt/mem}/alias_alloc.lua (53%)
>   create mode 100644 test/LuaJIT-tests/opt/mem/index
>   rename test/LuaJIT-tests/{misc/gc_trace.lua => trace/gc.lua} (54%)
>   create mode 100644 test/LuaJIT-tests/trace/hook_norecord.lua
>   create mode 100644 test/LuaJIT-tests/trace/hook_record.lua
>   create mode 100644 test/LuaJIT-tests/trace/jit_flush.lua
>   rename test/LuaJIT-tests/{misc/phi_conv.lua => trace/phi/conv.lua} (65%)
>   rename test/LuaJIT-tests/{misc => trace}/stack_purge.lua (74%)
>   rename test/LuaJIT-tests/{misc => trace}/tcall_base.lua (55%)
>   create mode 100644 test/LuaJIT-tests/trace/tcall_loop.lua
>   create mode 100644 test/LuaJIT-tests/trace/unordered_jit.lua
>   create mode 100644 test/LuaJIT-tests/trace/wbarrier.lua
>

[-- Attachment #2: Type: text/html, Size: 10811 bytes --]