Sergey,
thanks for the patch! LGTM
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 }}