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

[Maxim Kokryashkin]

Hi, Igor!
Thanks for the fixes!
LGTM
--
Best regards,
Maxim Kokryashkin
 
 
> 
>>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
> 
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.tarantool.org/pipermail/tarantool-patches/attachments/20230301/2d43671e/attachment.htm>