Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Automatically run a profiler with the executable #348

Merged
merged 4 commits into from
Dec 3, 2024
Merged

Conversation

giordano
Copy link
Member

@giordano giordano commented Oct 28, 2024

Proof-of-concept for #347, at the moment only supporting vtune. Everything is up-to-discussion, this is just to show how this could work.

You can run this for the stream example with

reframe -c benchmarks/examples/stream -r --performance-report -S profiler=vtune

(the main point is the option -S profiler=vtune, the rest are whatever other options you want to pass to reframe).

@giordano giordano force-pushed the mg/profiler branch 4 times, most recently from 61dc564 to 55d7fb7 Compare November 4, 2024 15:41
@giordano giordano force-pushed the mg/profiler branch 3 times, most recently from 14eb32d to 7491869 Compare November 13, 2024 18:33
@giordano giordano force-pushed the mg/profiler branch 3 times, most recently from 727ed48 to 8816745 Compare November 28, 2024 15:17
@giordano giordano force-pushed the mg/profiler branch 2 times, most recently from 024a9d4 to 93f980c Compare November 29, 2024 13:15
@giordano giordano marked this pull request as ready for review December 3, 2024 12:00
@giordano giordano requested review from tkoskela and ilectra December 3, 2024 12:00
Copy link
Member

@tkoskela tkoskela left a comment

Choose a reason for hiding this comment

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

LGTM!

@tkoskela tkoskela merged commit 610c241 into main Dec 3, 2024
6 checks passed
@tkoskela tkoskela deleted the mg/profiler branch December 3, 2024 12:30
github-actions bot pushed a commit that referenced this pull request Dec 3, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants