-
Notifications
You must be signed in to change notification settings - Fork 5k
[Perf] Add build time measurements for apple mobile and android scenarios #113469
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.
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
Conversation
Tagging subscribers to this area: @akoeplinger, @matouskozak |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates the runtime-perf pipeline to capture build logs for reporting build times, which will help identify AOT compiler regressions.
- Updated build scripts to redirect output to a build.log file and then echo its content.
- Added new PublishBuildArtifacts tasks to upload the captured build logs for various configurations (Android Mono, Android CoreCLR, iOS Mono, and iOS-NativeAOT).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me 👍.
Description
This is a follow-up of dotnet/performance#4842.
This PR adds build time measurements for apple mobile and android scenarios.
Validation
Run the internal dotnet-runtime-perf tool to verify that binlog files are uploaded and build time measurements are collected.
Contributes to #113073