Re: [Tarantool-patches] [PATCH luajit 02/36] test: enable <misc/alias_alloc.lua> LuaJIT test

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

Sergey,

On 15.08.24, Sergey Bronnikov wrote:
> Hi, Sergey
> 
> Thanks for the patch! See my comments below.
> 
> On 14.08.2024 16:55, Sergey Kaplun wrote:
> > This patch moves the <alias_alloc.lua> test from the <misc> to the
> > created <opt/mem> directory, includes it in <index>, and names subtests.
> >
> > Testing optimizations in <lj_opt_mem.c> requires CSE, DCE, FOLD, and FWD
> > optimizations enabled. Hence, the corresponding flags are used for the
> > <mem> directory in <index>.
> 
> It is still unclear for me *why* a reason of this change.

The tests for other optimizations also require the necessary flags, see
<index>. Also, I don't understand what *change* you mean -- it is
enabling of test, not changing its functionality.

> 

<snipped>

-- 
Best regards,
Sergey Kaplun