[Tarantool-patches] [PATCH luajit] test: add test for conversions folding
Maxim Kokryashkin
m.kokryashkin at tarantool.org
Fri Nov 17 13:51:09 MSK 2023
Hi, Sergey!
Thanks for the patch!
LGTM, except for the single nit regarding the commit message.
On Thu, Nov 16, 2023 at 04:44:56PM +0300, Sergey Kaplun wrote:
> This patch adds the test for commit
> 1a401622fe83ae695226c94df3416e312315d659 ("Fix assertions."). This patch
> removes incorrect assertions in the fold optimizations for conversions
> from numbers to different integer types. Although the issue affects only
Typo: s/Although/Since/
> branch 2.0, there is no need to fix it. Nevertheless, the test is
> required to avoid regressions in the future.
>
> Part of tarantool/tarantool#9145
> ---
>
> Branch: https://github.com/tarantool/luajit/tree/skaplun/lj-833-fold-conv-from-num
> Tarantool PR: https://github.com/tarantool/tarantool/pull/9375
> Related issues:
> * https://github.com/LuaJIT/LuaJIT/issues/833
> * https://github.com/tarantool/tarantool/issues/9145
>
> .../lj-833-fold-conv-from-num.test.lua | 33 +++++++++++++++++++
> 1 file changed, 33 insertions(+)
> create mode 100644 test/tarantool-tests/lj-833-fold-conv-from-num.test.lua
<snipped>
More information about the Tarantool-patches
mailing list