Skip to content

Conversation

@PatKamin
Copy link
Contributor

@PatKamin PatKamin commented Aug 22, 2025

  • stabilize results with a small sleep between binary runs
  • add combo profiler functionality which allows for choosing between time and cpu count measurement

@PatKamin PatKamin force-pushed the bump-compute-benchmarks branch from 54505c2 to 505b727 Compare August 25, 2025 12:41
@PatKamin PatKamin marked this pull request as ready for review August 25, 2025 12:42
@PatKamin PatKamin requested a review from a team as a code owner August 25, 2025 12:42
@PatKamin PatKamin force-pushed the bump-compute-benchmarks branch from 505b727 to a94c5af Compare August 25, 2025 14:23
@PatKamin PatKamin requested a review from mateuszpn August 26, 2025 07:42
Copy link
Contributor

@mateuszpn mateuszpn left a comment

Choose a reason for hiding this comment

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

lgtm

- stabilize results with a small sleep between binary runs
- add combo profiler functionality to allow for choosing between time
and cpu count measurement
These scenarios where meant to be ran on BMG systems, not on PVC.
@PatKamin
Copy link
Contributor Author

PatKamin commented Sep 4, 2025

@intel/llvm-gatekeepers, this is ready to merge. CI failures are unrelated to changes in benchmarks framework.

@PatKamin
Copy link
Contributor Author

PatKamin commented Sep 8, 2025

@intel/llvm-gatekeepers, a friendly ping

@uditagarwal97
Copy link
Contributor

@PatKamin Approval from @intel/llvm-reviewers-benchmarking is missing.

@ianayl
Copy link
Contributor

ianayl commented Sep 8, 2025

Benchmarking scripts look good, and passes precommit testing: https://github.com/intel/llvm/actions/runs/17434257703/job/49506851128?pr=19857

Going to wait on https://github.com/intel/llvm/actions/runs/17557291851/job/49864616578 to assess impact on CI

Copy link
Contributor

@ianayl ianayl left a comment

Choose a reason for hiding this comment

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

benchmark scripts lgtm

@PatKamin
Copy link
Contributor Author

@intel/llvm-gatekeepers, please merge. CI failures are unrelated, required approval granted.

@uditagarwal97 uditagarwal97 merged commit 0eb7c73 into intel:sycl Sep 12, 2025
41 of 44 checks passed
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.

5 participants