[Tarantool-patches] [PATCH v2 luajit 29/41] perf: adjust scimark-fft in LuaJIT-benches

[Sergey Kaplun]

Hi, Sergey!
Thanks for the review!
Please consider my answer below.

On 02.01.26, Sergey Bronnikov wrote:
> Hi, Sergey,
> 
> thanks for the patch! LGTM with a minor comment.
> 
> Sergey
> 
> On 12/26/25 12:18, Sergey Kaplun wrote:
> > This patch adjusts the aforementioned test to use the benchmark
> > framework introduced before. The default arguments are adjusted
> > according to the <PARAM_x86.txt> file. The arguments to the script still
> > can be provided in the command line run.
> >
> > Checks are omitted since they were not present in the original suite,
> > plus the precise result value depends on the input parameter.
> > ---
> >   perf/LuaJIT-benches/scimark-fft.lua | 19 ++++++++++++++++++-
> >   1 file changed, 18 insertions(+), 1 deletion(-)
> >
> > diff --git a/perf/LuaJIT-benches/scimark-fft.lua b/perf/LuaJIT-benches/scimark-fft.lua
> > index c05bb69a..96535774 100644
> > --- a/perf/LuaJIT-benches/scimark-fft.lua
> > +++ b/perf/LuaJIT-benches/scimark-fft.lua
> > @@ -1 +1,18 @@
> The small test description was not added. Is it intentional?

Yes, see my answer in the previous series.
The description and main payload are in the <scimark_lib.lua> file.
I see no reason for duplication.

> > -require("scimark_lib").FFT(1024)(tonumber(arg and arg[1]) or 50000)
> > +local bench = require("bench").new(arg)
> > +
> > +local cycles = tonumber(arg and arg[1]) or 50000
> > +local benchmark
> > +benchmark = {
> > +  name = "scimark_fft",
> > +  -- XXX: The description of tests for the function is too
> > +  -- inconvenient.
> > +  skip_check = true,
> > +  payload = function()
> > +    local flops = require("scimark_lib").FFT(1024)(cycles)
> > +    benchmark.items = flops
> > +  end,
> > +}
> > +
> > +bench:add(benchmark)
> > +
> > +bench:run_and_report()

-- 
Best regards,
Sergey Kaplun