<!DOCTYPE html>
<html data-lt-installed="true">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body style="padding-bottom: 1px;">
<p>Sergey,</p>
<p>thanks for the patch! LGTM<br>
</p>
<div class="moz-cite-prefix">On 22.04.2024 11:49, Sergey Kaplun
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:400b3ae2c1454663c6d975b55a68db2621132cc1.1713773432.git.skaplun@tarantool.org">
<pre class="moz-quote-pre" wrap="">This flavor enables the LUAJIT_ENABLE_TABLE_BUMP option to test table
bump optimization.
Needed for tarantool/tarantool#9924
---
.github/workflows/exotic-builds-testing.yml | 10 +++++++++-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/.github/workflows/exotic-builds-testing.yml b/.github/workflows/exotic-builds-testing.yml
index 859603bd..6ce100f4 100644
--- a/.github/workflows/exotic-builds-testing.yml
+++ b/.github/workflows/exotic-builds-testing.yml
@@ -34,7 +34,7 @@ jobs:
BUILDTYPE: [Debug, Release]
ARCH: [ARM64, x86_64]
GC64: [ON, OFF]
- FLAVOR: [checkhook, dualnum, gdbjit, nojit, nounwind]
+ FLAVOR: [checkhook, dualnum, gdbjit, nojit, nounwind, tablebump]
include:
- BUILDTYPE: Debug
CMAKEFLAGS: -DCMAKE_BUILD_TYPE=Debug -DLUA_USE_ASSERT=ON -DLUA_USE_APICHECK=ON
@@ -50,12 +50,20 @@ jobs:
FLAVORFLAGS: -DLUAJIT_USE_GDBJIT=ON
- FLAVOR: nounwind
FLAVORFLAGS: -DLUAJIT_NO_UNWIND=ON
+ - FLAVOR: tablebump
+ FLAVORFLAGS: -DLUAJIT_ENABLE_TABLE_BUMP=ON
exclude:
- ARCH: ARM64
GC64: OFF
# DUALNUM is default for ARM64, no need for additional testing.
- FLAVOR: dualnum
ARCH: ARM64
+ # Within table bump optimization enabled (and due to our modification
+ # related to metrics), some offsets in GG_State stop fit in 12bit
+ # immediate. Hence, the build failed due to the DASM error
+ # (DASM_S_RANGE_I).
+ - FLAVOR: tablebump
+ ARCH: ARM64
runs-on: [self-hosted, regular, Linux, '${{ matrix.ARCH }}']
name: >
LuaJIT ${{ matrix.FLAVOR }}
</pre>
</blockquote>
</body>
<lt-container></lt-container>
</html>