Skip to content

[Benches] Add option to skip oneapi update#21194

Merged
kswiecicki merged 5 commits intointel:syclfrom
PatKamin:no-download-oneapi
Feb 11, 2026
Merged

[Benches] Add option to skip oneapi update#21194
kswiecicki merged 5 commits intointel:syclfrom
PatKamin:no-download-oneapi

Conversation

@PatKamin
Copy link
Contributor

@PatKamin PatKamin commented Feb 2, 2026

No description provided.

@PatKamin PatKamin requested a review from a team as a code owner February 2, 2026 15:40
Copy link
Contributor

@lukaszstolarczuk lukaszstolarczuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do you have an example CI run with this flag enabled?

Extend the scope of a former `--no-rebuild` argument:
- no benchmarks rebuilds
- no oneAPI download
- no benchmark data download (models, i.e.)
@PatKamin
Copy link
Contributor Author

PatKamin commented Feb 6, 2026

do you have an example CI run with this flag enabled?

An example run: https://github.com/intel/llvm/actions/runs/21753460244

Copy link
Contributor

@lukaszstolarczuk lukaszstolarczuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks promising, thx. Just one last comment

@PatKamin
Copy link
Contributor Author

This run finally looks ok: https://github.com/intel/llvm/actions/runs/21824066274/job/62964905039#step:20:1192
I've made one more small change after running this check: moved utils.download methods' offline skip message to debug.

@github-actions
Copy link
Contributor

@intel/llvm-gatekeepers please consider merging

@kswiecicki kswiecicki merged commit 7986f8c into intel:sycl Feb 11, 2026
13 checks passed
@PatKamin PatKamin deleted the no-download-oneapi branch February 11, 2026 15:09
luszczewskakasia1 pushed a commit to luszczewskakasia1/llvm that referenced this pull request Feb 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants