From c9c774f53ec23fb2b3c4950688b051d8f140c461 Mon Sep 17 00:00:00 2001 From: "Sarnie, Nick" Date: Wed, 30 Jul 2025 09:07:59 -0700 Subject: [PATCH 1/2] [CI] Fix Linux OneAPI build LIT tests Signed-off-by: Sarnie, Nick --- .github/workflows/sycl-linux-build.yml | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/.github/workflows/sycl-linux-build.yml b/.github/workflows/sycl-linux-build.yml index 0dcd828aefe14..b7069e03d9ba0 100644 --- a/.github/workflows/sycl-linux-build.yml +++ b/.github/workflows/sycl-linux-build.yml @@ -188,8 +188,10 @@ jobs: run: cmake --build $GITHUB_WORKSPACE/build --target ${{ inputs.build_target || 'sycl-toolchain' }} - name: check-llvm if: always() && !cancelled() && contains(inputs.changes, 'llvm') + env: + BUILD_CONFIGURE_EXTRA_ARGS: ${{ inputs.build_configure_extra_args }} run: | - if [[ "${{ inputs.build_configure_extra_args }}" == *"--use-libcxx"* ]]; then + if [[ "${BUILD_CONFIGURE_EXTRA_ARGS}" == *"--use-libcxx"* ]]; then # https://github.com/llvm/llvm-project/issues/59429 export LIT_FILTER_OUT="ExecutionEngine/MCJIT" fi @@ -197,10 +199,12 @@ jobs: cmake --build $GITHUB_WORKSPACE/build --target check-llvm - name: check-clang if: always() && !cancelled() && contains(inputs.changes, 'clang') + env: + BUILD_CONFIGURE_EXTRA_ARGS: ${{ inputs.build_configure_extra_args }} run: | # Can we move this to Dockerfile? Hopefully, noop on Windows. export XDG_CACHE_HOME=$GITHUB_WORKSPACE/os_cache - if [[ "${{ inputs.build_configure_extra_args }}" == *"--use-libcxx"* ]]; then + if [[ "${BUILD_CONFIGURE_EXTRA_ARGS}" == *"--use-libcxx"* ]]; then # https://github.com/llvm/llvm-project/issues/59428 export LIT_FILTER_OUT="(E|e)xception" fi From 255d0ad18f1ceabecab876d2d5d995d1c2b17c16 Mon Sep 17 00:00:00 2001 From: "Sarnie, Nick" Date: Wed, 30 Jul 2025 10:14:21 -0700 Subject: [PATCH 2/2] address feedback Signed-off-by: Sarnie, Nick --- .github/workflows/sycl-linux-build.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.github/workflows/sycl-linux-build.yml b/.github/workflows/sycl-linux-build.yml index b7069e03d9ba0..ec8a17b997bdd 100644 --- a/.github/workflows/sycl-linux-build.yml +++ b/.github/workflows/sycl-linux-build.yml @@ -189,6 +189,7 @@ jobs: - name: check-llvm if: always() && !cancelled() && contains(inputs.changes, 'llvm') env: + # Can't inline to support possible quotes inside: BUILD_CONFIGURE_EXTRA_ARGS: ${{ inputs.build_configure_extra_args }} run: | if [[ "${BUILD_CONFIGURE_EXTRA_ARGS}" == *"--use-libcxx"* ]]; then @@ -200,6 +201,7 @@ jobs: - name: check-clang if: always() && !cancelled() && contains(inputs.changes, 'clang') env: + # Can't inline to support possible quotes inside: BUILD_CONFIGURE_EXTRA_ARGS: ${{ inputs.build_configure_extra_args }} run: | # Can we move this to Dockerfile? Hopefully, noop on Windows.