[Tarantool-patches] [PATCH luajit v1 2/5] ci: add tablebump flavor for exotic builds
Sergey Bronnikov
sergeyb at tarantool.org
Thu Jun 6 13:56:30 MSK 2024
Sergey,
thanks for the patch! LGTM
On 22.04.2024 11:49, Sergey Kaplun wrote:
> 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 }}
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.tarantool.org/pipermail/tarantool-patches/attachments/20240606/c47d191e/attachment.htm>
More information about the Tarantool-patches
mailing list