[Tarantool-patches] [PATCH v1 luajit 33/41] perf: adjust scimark-sparse in LuaJIT-benches

[Sergey Bronnikov]

Hi, Sergey,

thanks for the patch! LGTM with a minor comment.

please add a small test description to the comment.

Sergey

On 10/24/25 13:50, 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-sparse.lua | 20 +++++++++++++++++++-
>   1 file changed, 19 insertions(+), 1 deletion(-)
>
> diff --git a/perf/LuaJIT-benches/scimark-sparse.lua b/perf/LuaJIT-benches/scimark-sparse.lua
> index 01a2258d..a855cc22 100644
> --- a/perf/LuaJIT-benches/scimark-sparse.lua
> +++ b/perf/LuaJIT-benches/scimark-sparse.lua
> @@ -1 +1,19 @@
> -require("scimark_lib").SPARSE(1000, 5000)(tonumber(arg and arg[1]) or 150000)
> +local bench = require("bench").new(arg)
> +
> +local cycles = tonumber(arg and arg[1]) or 150000
> +
> +local benchmark
> +benchmark = {
> +  name = "scimark_sparse",
> +  -- XXX: The description of tests for the function is too
> +  -- inconvenient.
> +  skip_check = true,
> +  payload = function()
> +    local flops = require("scimark_lib").SPARSE(1000, 5000)(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/20251117/14544fab/attachment.htm>