tarantool / luajit
93%
tarantool/master: 93%

LAST BUILD BRANCH: tarantool/release/3.6
DEFAULT BRANCH: tarantool/master
Repo Added
Files 89
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH tarantool/release/3.3
branch: tarantool/release/3.3
CHANGE BRANCH
x

coverage: 93.048% (-0.02%) from 93.065%
19798590866

push

github

Buristan 
ARM64: Fix assembly of HREFK (again).

Thanks to Peter Cawley.

(cherry picked from commit 93ce12ee1)

When assembling the HREFK IR with the huge offset of the target node
from the table, this offset calculation and the key loading from the
node are emitted like the following:
|  ldr   x16, [x2, 40]
|  add   x16, x16, x21
|  ldr   x27, [x16, 8]
|  cmp   x27, x17

Here, `x16` is the node register, `x27` is the key register, and `x21`
is the register containing the offset.

It is possible that the register for holding the constant operand in the
addition may be chosen as the same register containing the node address,
since the full `RSET_GPR` is given to the `emit_opk()`. It will result
in the following invalid mcode:
|  ldr   x27, [x2, 40]
|  str   x27, [sp, 8]
|  add   x16, x16, x16
|  ldr   x16, [sp, 8]
|  ldr   x27, [x16, 8]
|  cmp   x27, x17

It seems that in the current implementation the LuaJIT's register
allocator always prefers the register holding the key instead, so this
does not lead to the invalid emitting. Hence, it is impossible to come
up with any valid reproducer. However, to avoid possible regressions in
the future, this patch fixes the invalid register set by excluding the
node register from it.

Sergey Kaplun:
* added the description for the problem

Part of tarantool/tarantool#11691

Reviewed-by: Sergey Bronnikov <sergeyb@tarantool.org>
Signed-off-by: Sergey Kaplun <skaplun@tarantool.org>
(cherry picked from commit fd5e7cb52)

5712 of 6046 branches covered (94.48%)

Branch coverage included in aggregate %.

21792 of 23513 relevant lines covered (92.68%)

3873117.69 hits per line

Relevant lines Covered
23513 RELEVANT LINES 21792 COVERED LINES
3873117.69 HITS PER LINE
Source Files on tarantool/release/3.3

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
19798590866 tarantool/release/3.3 ARM64: Fix assembly of HREFK (again). Thanks to Peter Cawley. (cherry picked from commit 93ce12ee1) When assembling the HREFK IR with the huge offset of the target node from the table, this offset calculation and ... push Buristan github
93.05
19796510619 tarantool/release/3.3 macOS: Remove obsolete -single_module flag. Thanks to dundargoc. (cherry picked from commit 97813fb92) The aforementioned flag is the default since Xcode 2.2 [1]. This patch removes this flag to avoid warnings for... push Buristan github
93.06
17649971406 tarantool/release/3.3 Invalidate SCEV entry when returning to lower frame. Thanks to Zhongwei Yao. (cherry picked from commit 65c849390) When returning to the lower frame, LuaJIT does not clear the Scalar Evolution analysis entry. Henc... push Buristan github
93.06
17645235118 tarantool/release/3.3 ARM64: Fix LDP/STP fusion (again). Reported and analyzed by Zhongwei Yao. Fix by Peter Cawley. (cherry picked from commit b8c6ccd50) Assume we have stores/loads from the pointer with offset +488 and -16. The lower... push Buristan github
93.06
17640023344 tarantool/release/3.3 x86/x64: Don't use undefined MUL/IMUL zero flag. Reported by VrIgHtEr. (cherry picked from commit c92d0cb19) When emitting the arithmetic operations on registers via the `asm_intarith()`, the next `test` instructi... push Buristan github
93.03
17636771727 tarantool/release/3.3 ci: drop CentOS 7 Tarantool integration workflow The corresponding workflow is going to be removed from Tarantool[^1]. [^1]: https://github.com/tarantool/tarantool/pull/11396 Reviewed-by: Sergey Kaplun <skaplun@tarantool.org> Reviewed-by: Serge... push Buristan github
93.07
17634222921 tarantool/release/3.3 Correctly close VM state after early OOM during open. Reported by Assumeru. (cherry picked from commit 5ca25ee83) `lua_newstate()` sets `g->str.mask` to `~(MSize)0` before calling `cpluaopen(). If OOM happens befo... push Buristan github
93.06
17618265815 tarantool/release/3.3 FFI: Fix dangling CType references (again). Reported by Sergey Kaplun. (cherry picked from commit c64020f3c) This patch fixes JIT recording for the vararg calls, that was not fixed in the previous patch. The `ctr`... push Buristan github
93.11
17614139297 tarantool/release/3.3 Avoid out-of-range PC for stack overflow error from snapshot restore. Reported by Sergey Kaplun. (cherry picked from commit e3fa3c48d) In case when the saved PC in the snapshot is the first (0th index) PC in the p... push Buristan github
93.05
17609805760 tarantool/release/3.3 ARM64: Prevent STP fusion for conditional code emitted by TBAR. Thanks to Peter Cawley. (cherry picked from commit 7cc53f0b8) Assume we have a trace for the several `setmetatable()` calls to the same table. This t... push Buristan github
93.1
See All Builds (1911)