Migrate to compiler-research/ci-workflows #6
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,92 @@ | ||
| name: Nightly | ||
|
|
||
| # Exhaustive run on a schedule: the full matrix with the xfail sweeps and | ||
| # valgrind on every Linux cell, against the pinned CppInterOp (reusable | ||
| # default). On any failure a tracking issue is opened/updated. To restore | ||
| # upstream-drift detection, add a cell with `cppinterop-ref: main` (will be | ||
| # red until the forks are re-synced to CppInterOp's strong-types API). | ||
|
|
||
| on: | ||
| schedule: | ||
| - cron: '30 2 * * *' | ||
| workflow_dispatch: | ||
|
|
||
| permissions: | ||
| contents: read | ||
|
|
||
| concurrency: | ||
| group: nightly-${{ github.workflow }} | ||
| cancel-in-progress: true | ||
|
|
||
| jobs: | ||
| build-test: | ||
| name: ${{ matrix.os }}/llvm${{ matrix.llvm }}/py${{ matrix.py }}/c++${{ matrix.cxx }}${{ matrix.vg && '/vg' || '' }}${{ matrix.cling && '/cling' || '' }} | ||
| strategy: | ||
| fail-fast: false | ||
| matrix: | ||
| # Superset of the PR matrix + breadth (py 3.12/3.13, C++17, LLVM 20, arm) | ||
| # and one cling cell. valgrind only on LLVM 21 cells: vendored etc/ has | ||
| # clang21 supps only, so other majors would surface unsuppressed noise. | ||
| include: | ||
| - { os: ubuntu-24.04, llvm: '20', flavor: system, py: '3.12', cxx: '17', vg: false } | ||
| - { os: ubuntu-24.04, llvm: '21', flavor: system, py: '3.13', cxx: '20', vg: true } | ||
| - { os: ubuntu-24.04, llvm: '21', flavor: system, py: '3.14', cxx: '20', vg: true } | ||
| - { os: ubuntu-24.04, llvm: '22', flavor: '', py: '3.14', cxx: '20', vg: false } | ||
| - { os: ubuntu-24.04, llvm: '21', flavor: system, py: '3.12', cxx: '20', vg: true } | ||
| - { os: ubuntu-24.04-arm, llvm: '21', flavor: system, py: '3.13', cxx: '20', vg: true } | ||
| - { os: ubuntu-24.04-arm, llvm: '20', flavor: system, py: '3.12', cxx: '17', vg: false } | ||
| - { os: macos-26, llvm: '21', flavor: system, py: '3.12', cxx: '20', vg: false } | ||
| - { os: macos-26-intel, llvm: '21', flavor: system, py: '3.12', cxx: '20', vg: false } | ||
| # cling backend, against the cached llvm-root (cling-llvm20) recipe cell. | ||
| - { os: ubuntu-24.04, llvm: '20', flavor: cling, fv: cling-llvm20, cling: true, py: '3.13', cxx: '20', vg: false } | ||
| uses: aaronj0/ci-workflows/.github/workflows/cppjit.yml@cppjit-reusable | ||
| # NOTE: retarget to compiler-research/ci-workflows/...@main once upstreamed. | ||
| with: | ||
| os: ${{ matrix.os }} | ||
| llvm-version: ${{ matrix.llvm }} | ||
| llvm-flavor: ${{ matrix.flavor }} | ||
| llvm-flavor-version: ${{ matrix.fv || '' }} | ||
| python-version: ${{ matrix.py }} | ||
| cxx-standard: ${{ matrix.cxx }} | ||
| valgrind: ${{ matrix.vg }} | ||
| run-sweeps: true | ||
|
Contributor
There was a problem hiding this comment. This name makes no sense -- maybe run-all-crashing-tests or something is better. |
||
| cling: ${{ matrix.cling || false }} | ||
|
|
||
| drift-issue: | ||
| needs: build-test | ||
| if: ${{ always() && needs.build-test.result == 'failure' }} | ||
| runs-on: ubuntu-latest | ||
| permissions: | ||
| contents: read | ||
| issues: write | ||
| steps: | ||
| - uses: actions/github-script@v7 | ||
| with: | ||
| script: | | ||
| const marker = '<!-- cppjit-nightly-drift -->'; | ||
| const run = `${context.serverUrl}/${context.repo.owner}/${context.repo.repo}/actions/runs/${context.runId}`; | ||
| const body = [ | ||
| marker, | ||
| '## Nightly build/test failed', | ||
| '', | ||
| `The nightly run (CppJIT \`master\` against CppInterOp \`main\`) failed.`, | ||
| `Most likely an upstream CppInterOp drift; investigate and sync if needed.`, | ||
| '', | ||
| `Run: ${run}`, | ||
| `Date: ${new Date().toISOString()}`, | ||
| ].join('\n'); | ||
|
|
||
| const existing = await github.paginate(github.rest.issues.listForRepo, { | ||
| ...context.repo, state: 'open', labels: 'nightly-drift', | ||
| }); | ||
| const hit = existing.find(i => (i.body || '').startsWith(marker)); | ||
| if (hit) { | ||
| await github.rest.issues.createComment({ ...context.repo, issue_number: hit.number, body }); | ||
| } else { | ||
| await github.rest.issues.create({ | ||
| ...context.repo, | ||
| title: 'Nightly: CppJIT broken against CppInterOp main', | ||
| labels: ['nightly-drift'], | ||
| body, | ||
| }); | ||
| } | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,76 @@ | ||
| # Configures the CppInterOp ExternalProject. Default backend is clang-repl; | ||
|
Contributor
There was a problem hiding this comment. Why do we need this file? If it has something to do with the ci, then then it defeats the purpose of the ci-workflows changes. |
||
| # CPPJIT_USE_CLING builds CppInterOp against a prebuilt cling instead (for | ||
| # ROOT, cppjit does NOT build cling itself, it accepts a build). | ||
| include_guard(GLOBAL) | ||
| include(ExternalProject) | ||
|
|
||
| # Developer toggle: Cling from either ROOT or standalone supplies LLVM_DIR/Clang_DIR | ||
| option(CPPJIT_USE_CLING "Build CppInterOp against a prebuilt Cling C++ Interpreter (ROOT)" OFF) | ||
| mark_as_advanced(CPPJIT_USE_CLING) | ||
|
|
||
| set(Cling_DIR "" CACHE PATH "Cling CMake config dir; derived from LLVM_DIR when empty (cling mode only)") | ||
| # Surface Cling_DIR in ccmake/GUI only when cling mode is on. | ||
| if(CPPJIT_USE_CLING) | ||
| mark_as_advanced(CLEAR Cling_DIR) | ||
| else() | ||
| mark_as_advanced(Cling_DIR) | ||
| endif() | ||
|
|
||
| function(cppjit_add_cppinterop) | ||
| set(_args | ||
| -DLLVM_DIR=${LLVM_DIR} | ||
| -DCPPINTEROP_ENABLE_TESTING=${CPPJIT_ENABLE_CPPINTEROP_TESTS} | ||
| -DBUILD_SHARED_LIBS=ON | ||
| -DCMAKE_INSTALL_PREFIX=${CPPINTEROP_INSTALL_DIR} | ||
| -DCMAKE_INSTALL_LIBDIR=lib | ||
| -DCMAKE_BUILD_TYPE=${CMAKE_BUILD_TYPE} | ||
| -DCMAKE_CXX_STANDARD=17 | ||
| ) | ||
|
|
||
| if(CPPJIT_USE_CLING) | ||
| set(_cling_dir "${Cling_DIR}") | ||
| if(NOT _cling_dir) | ||
| # LLVM_DIR is <prefix>/lib/cmake/llvm; cling config sits at the | ||
| # sibling <prefix>/lib/cmake/cling (setup-llvm flavor:cling layout). | ||
| get_filename_component(_prefix "${LLVM_DIR}" DIRECTORY) # <prefix>/lib/cmake | ||
| get_filename_component(_prefix "${_prefix}" DIRECTORY) # <prefix>/lib | ||
| get_filename_component(_prefix "${_prefix}" DIRECTORY) # <prefix> | ||
| set(_cling_dir "${_prefix}/lib/cmake/cling") | ||
| endif() | ||
| message(STATUS "CppInterOp backend: cling (Cling_DIR=${_cling_dir})") | ||
| list(APPEND _args | ||
| -DCPPINTEROP_USE_CLING=ON | ||
| -DCPPINTEROP_USE_REPL=OFF | ||
| -DCling_DIR=${_cling_dir} | ||
| ) | ||
| else() | ||
| list(APPEND _args | ||
| -DCPPINTEROP_USE_REPL=ON | ||
| -DCPPINTEROP_USE_CLING=OFF | ||
| ) | ||
| endif() | ||
|
|
||
| if(Clang_DIR) | ||
| list(APPEND _args -DClang_DIR=${Clang_DIR}) | ||
| endif() | ||
| if(CMAKE_C_COMPILER) | ||
| list(APPEND _args -DCMAKE_C_COMPILER=${CMAKE_C_COMPILER}) | ||
| endif() | ||
| if(CMAKE_CXX_COMPILER) | ||
| list(APPEND _args -DCMAKE_CXX_COMPILER=${CMAKE_CXX_COMPILER}) | ||
| endif() | ||
|
|
||
| ExternalProject_Add(CppInterOp | ||
| GIT_REPOSITORY https://github.com/compiler-research/CppInterOp.git | ||
| GIT_TAG ${CPPINTEROP_GIT_TAG} | ||
| PREFIX "${CMAKE_BINARY_DIR}/CppInterOp" | ||
| CMAKE_ARGS ${_args} | ||
| BUILD_BYPRODUCTS | ||
| "${CPPINTEROP_INSTALL_DIR}/lib/libclangCppInterOp${CMAKE_SHARED_LIBRARY_SUFFIX}" | ||
| LOG_DOWNLOAD ON | ||
| LOG_CONFIGURE ON | ||
| LOG_BUILD ON | ||
| LOG_INSTALL ON | ||
| LOG_OUTPUT_ON_FAILURE ON | ||
| ) | ||
| endfunction() | ||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need
name:here because it duplicates the strings and in the github ci bar it shows twice.