Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Add hotspot analysis using perf #8

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Add hotspot analysis using perf #8

wants to merge 1 commit into from

Conversation

m-kahla
Copy link

@m-kahla m-kahla commented Jun 12, 2021 •
edited
Loading

Added functionality to benchmark the builds using perf and display in the reports

@ggeorgakoudis ggeorgakoudis changed the title Perf hotness Add hotspot analysis using perf Jun 24, 2021
ggeorgakoudis
ggeorgakoudis requested changes Jun 24, 2021
View reviewed changes
Copy link
Member

@ggeorgakoudis ggeorgakoudis left a comment

Choose a reason for hiding this comment

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

Diff shows a lot of changes that are only about formatting. Can you please revert those changes since they are unrelated to the functionality added?

Use 'env' as a separate key that has the env variables in the input Y…
30f4dd1 
…AML file

Add new option to run experiments with perf and get the hotness of each line of code
Add Hotness Columns for each build in HTML reports of Source Code
Only show Hotness coulmn in the reports if we run with perf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ggeorgakoudis ggeorgakoudis ggeorgakoudis requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@m-kahla @ggeorgakoudis