tarantool / luajit / 11011991618

93%

tarantool/master: 93%

Build Type

push

github

Committed by Buristan

Commit Message

Add missing coercion when recording select(string, ...)

Thanks to Peter Cawley.

(cherry picked from commit 92b89d005)

Before the patch, the recording of `select()` with a string argument
leads to the following IR:
| rcx   >  int CONV   "1"   int.num index
Where the operand has string type instead of number type.
This leads to the corresponding mcode:
| cvttsd2si ecx, xmm1
Where xmm1 has an undefined value. Thus leads to the undefined behaviour
for the recording trace.

This patch adds the missing coercion.

Sergey Kaplun:
* added the description and the test for the problem

Part of tarantool/tarantool#10199

Run Details

5688 of 6028 branches covered (94.36%)

Branch coverage included in aggregate %.

3 of 3 new or added lines in 1 file covered. (100.0%)

7 existing lines in 2 files now uncovered.

21683 of 23437 relevant lines covered (92.52%)

2915749.97 hits per line