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

[Sergey Bronnikov]

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?
> -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()
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.tarantool.org/pipermail/tarantool-patches/attachments/20260102/6c1e8b28/attachment.htm>