Skip to content

Commit ae74e38

Browse files
authored
[CI] Correctly set install_igc_driver in postcommit (#17488)
I forgot to do this when I updated the workflow to run on driver updates, so it wasn't using the new driver. --------- Signed-off-by: Sarnie, Nick <nick.sarnie@intel.com>
1 parent 64546e7 commit ae74e38

File tree

1 file changed

+12
-1
lines changed

1 file changed

+12
-1
lines changed

.github/workflows/sycl-post-commit.yml

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,10 @@ concurrency:
2929
permissions: read-all
3030

3131
jobs:
32+
detect_changes:
33+
if: ${{ github.event_name == 'pull_request' }}
34+
uses: ./.github/workflows/sycl-detect-changes.yml
35+
3236
build-lin:
3337
name: Linux (Self build + no-assertions)
3438
if: github.repository == 'intel/llvm'
@@ -40,7 +44,7 @@ jobs:
4044
build_configure_extra_args: --no-assertions --hip --cuda --native_cpu -DSYCL_ENABLE_STACK_PRINTING=ON -DSYCL_LIB_WITH_DEBUG_SYMBOL=ON
4145

4246
e2e-lin:
43-
needs: [build-lin]
47+
needs: [detect_changes, build-lin]
4448
if: ${{ always() && !cancelled() && needs.build-lin.outputs.build_conclusion == 'success' }}
4549
strategy:
4650
fail-fast: false
@@ -94,6 +98,13 @@ jobs:
9498
sycl_toolchain_archive: ${{ needs.build-lin.outputs.artifact_archive_name }}
9599
sycl_toolchain_decompress_command: ${{ needs.build-lin.outputs.artifact_decompress_command }}
96100

101+
# Do not install drivers on AMD and CUDA runners.
102+
install_igc_driver: >-
103+
${{ github.event_name == 'pull_request' }} &&
104+
${{ !contains(matrix.target_devices, 'cuda') &&
105+
!contains(matrix.target_devices, 'hip') &&
106+
contains(needs.detect_changes.outputs.filters, 'drivers') }}
107+
97108
build-win:
98109
if: |
99110
always()

0 commit comments

Comments
 (0)