Hi, Sergey!
Thanks for the patch!
LGTM
--
Best regards,
Maxim Kokryashkin
 
 
 
This commit adds the build with the aforementioned option to test
on x86_64 and arm64 architectures to exotic build matrix.
---

Branch: https://github.com/tarantool/luajit/tree/skaplun/gh-noticket-ci-checkhook
Workflow example: https://github.com/tarantool/luajit/actions/runs/4101136217

 .github/workflows/exotic-builds-testing.yml | 15 +++++++++++----
 1 file changed, 11 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/exotic-builds-testing.yml b/.github/workflows/exotic-builds-testing.yml
index cd0c14d7..aaf4f9f6 100644
--- a/.github/workflows/exotic-builds-testing.yml
+++ b/.github/workflows/exotic-builds-testing.yml
@@ -29,13 +29,14 @@ concurrency:
   cancel-in-progress: true
 
 jobs:
- test-x86_64-exotic:
+ test-exotic:
     strategy:
       fail-fast: false
       matrix:
         BUILDTYPE: [Debug, Release]
+ ARCH: [ARM64, x86_64]
         GC64: [ON, OFF]
- FLAVOR: [dualnum]
+ FLAVOR: [dualnum, checkhook]
         include:
           - BUILDTYPE: Debug
             CMAKEFLAGS: -DCMAKE_BUILD_TYPE=Debug -DLUA_USE_ASSERT=ON -DLUA_USE_APICHECK=ON
@@ -43,10 +44,16 @@ jobs:
             CMAKEFLAGS: -DCMAKE_BUILD_TYPE=RelWithDebInfo
           - FLAVOR: dualnum
             FLAVORFLAGS: -DLUAJIT_NUMMODE=2
- runs-on: [self-hosted, regular, Linux, x86_64]
+ - FLAVOR: checkhook
+ FLAVORFLAGS: -DLUAJIT_ENABLE_CHECKHOOK=ON
+ exclude:
+ # DUALNUM is default for ARM64, no need for additional testing.
+ - FLAVOR: dualnum
+ ARCH: ARM64
+ runs-on: [self-hosted, regular, Linux, '${{ matrix.ARCH }}']
     name: >
       LuaJIT ${{ matrix.FLAVOR }}
- (Linux/x86_64)
+ (Linux/${{ matrix.ARCH }})
       ${{ matrix.BUILDTYPE }}
       GC64:${{ matrix.GC64 }}
     steps:
--
2.34.1