[Tarantool-patches] [PATCH luajit 5/7] test: add skipcond for all JIT-related tests

Igor Munkin imun at tarantool.org
Tue Feb 28 22:02:12 MSK 2023


Max,

Thanks for your review! I've fixed the comments you've left.

On 16.02.23, Maxim Kokryashkin wrote:
> 
> Hi, Igor!
> Thanks for the patch!
> Please consider my comments below.
>  
> `lj-906-fix-err-mem.test.lua` has a `jit.off()`
> call in the line 78, which should be wrapped in a pcall.

Hm, thanks for noticing! Fun fact: jit.off doesn't raise the error when
JIT is not compiled. I guess we can left pcall only for jit.flush calls.

>  
> This change should be moved to commit
> «build: fix build with JIT disabled»
> (a835fb0f778db6e9f0109a66ea1d2ac78fe682e4)
> along with the change that I pointed out below. 

As we discussed offline, these changes should be left here. Ignoring.

>
<snipped>

> --
> Best regards,
> Maxim Kokryashkin
>  

-- 
Best regards,
IM


More information about the Tarantool-patches mailing list