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.

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

[release/7.0-staging] [MONO] Add JIT_CODE_DEBUG_INFO record functionality for Jitdump #87428

Merged

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Jun 12, 2023

Backport of #85107 to release/7.0-staging

/cc @fanyang-mono @saitama951

Customer Impact

Fixed jitdump version and added the support to annotate source code in perf when using Mono jitdump feature. A customer has reported this issue by dotnet/performance#2988

Testing

Manually tested by @saitama951 and passed CI testing.

Risk

None. This is an added capability for performance diagnostics on linux.

@carlossanlop
Copy link
Member

@fanyang-mono @lambdageek @vargaz Today is code complete for the July servicing release. Please get a Tactics approval, sign-off, confirm the CI failures are unrelated, and merge before EOD if you'd like this to be included in the release.

@fanyang-mono
Copy link
Member

@carlossanlop Thanks for your reminder. This isn't an urgent change. It could wait for the next round. :)

@fanyang-mono
Copy link
Member

Verified that CI failures are not related to this PR.

@fanyang-mono
Copy link
Member

Tactics has approved this.

@fanyang-mono fanyang-mono added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Jun 14, 2023
@fanyang-mono fanyang-mono merged commit 5e11a1d into release/7.0-staging Jun 14, 2023
@jkotas jkotas deleted the backport/pr-85107-to-release/7.0-staging branch June 16, 2023 23:40
@ghost ghost locked as resolved and limited conversation to collaborators Jul 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-VM-meta-mono Servicing-approved Approved for servicing release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants