Skip to content

Conversation

@jakobbotsch
Copy link
Member

The goal is to use these for automatic TP measurements in the
superpmi-asmdiffs pipeline, so they are not PGO optimized (required no
changes).

cc @dotnet/jit-contrib PTAL @BruceForstall

The goal is to use these for automatic TP measurements in the
superpmi-asmdiffs pipeline, so they are not PGO optimized (required no
changes).
@ghost
Copy link

ghost commented Apr 19, 2022

Tagging subscribers to this area: @hoyosjs
See info in area-owners.md if you want to be subscribed.

Issue Details

The goal is to use these for automatic TP measurements in the
superpmi-asmdiffs pipeline, so they are not PGO optimized (required no
changes).

cc @dotnet/jit-contrib PTAL @BruceForstall

Author: jakobbotsch
Assignees: jakobbotsch
Labels:

area-Infrastructure-coreclr

Milestone: -

@jakobbotsch jakobbotsch marked this pull request as draft April 19, 2022 19:31
@jakobbotsch jakobbotsch marked this pull request as ready for review April 19, 2022 20:30
Copy link
Contributor

@BruceForstall BruceForstall left a comment

Choose a reason for hiding this comment

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

LGTM.

However, I wouldn't merge this until we actually have code/automation in place, or ready, to use these new artifacts. It's obviously not free to produce them, so we don't want to waste the resources doing so unless we're going to use them.

@jakobbotsch
Copy link
Member Author

Gonna merge this now since #68292 will hopefully be merged very soon and I want to do a (hopefully final) run there to test, which needs a new release jit.

@jakobbotsch jakobbotsch merged commit 1d746d1 into dotnet:main Apr 22, 2022
@jakobbotsch jakobbotsch deleted the build-rolling-release-jits branch April 22, 2022 19:54
@ghost ghost locked as resolved and limited conversation to collaborators May 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants