tarantool / luajit / 6789043982

88%

tarantool/master: 93%

Build Type

push

github

Committed by igormunkin

Commit Message

Fix base register coalescing in side trace.

Thanks to Sergey Kaplun, NiLuJe and Peter Cawley.

(cherry-picked from commit aa2db7ebd)

The previous patch fixed just part of the problem with the register
coalescing. For example, the parent base register may be used inside the
parent or child register sets when it shouldn't. This leads to incorrect
register allocations, which may lead to crashes or undefined behaviour.
This patch fixes it by excluding the parent base register from both
register sets.

The test case for this patch doesn't fail before the commit since it
requires specific register allocation, which is hard to construct and
very fragile. Due to the lack of ideal sync with the upstream
repository, the test is passed before the patch.
It should become correct after backporting future patches.

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

Resolves tarantool/tarantool#8767
Part of tarantool/tarantool#9145

Run Details

5344 of 5974 branches covered (0.0%)

Branch coverage included in aggregate %.

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

13 existing lines in 1 file now uncovered.

20493 of 23302 relevant lines covered (87.95%)

2744052.66 hits per line