Skip to content

Conversation

@amanasifkhalid
Copy link
Contributor

Part of #107749. Stopping before morph seems like a sensible place, since maintaining the profile through morph will keep throw blocks with flow into them hot -- I expect that will incur plenty of diffs.

Waiting for #111006 and #111009 to go through first.

@ghost ghost added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Jan 3, 2025
@dotnet-policy-service
Copy link
Contributor

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

@amanasifkhalid
Copy link
Contributor Author

/azp run runtime-coreclr outerloop, Fuzzlyn, Antigen

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@amanasifkhalid amanasifkhalid marked this pull request as ready for review January 6, 2025 16:15
@amanasifkhalid
Copy link
Contributor Author

/azp run runtime-coreclr outerloop, Fuzzlyn, Antigen

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@amanasifkhalid
Copy link
Contributor Author

cc @dotnet/jit-contrib, @AndyAyersMS PTAL. Fuzzers didn't find any profile-related issues, and the outerloop runs are hitting #110958. Diffs are limited since we're missing some collections, though they're still fairly dramatic -- I suspect most of this churn is from tail-merging based on the diffs from #110656, and from the tweaks to fgFoldConditional since that's called in various phases. Diffs look to be largely in layout and LSRA.

Copy link
Member

@AndyAyersMS AndyAyersMS left a comment

Choose a reason for hiding this comment

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

Thanks!

Would be interested in seeing stats on how many methods are consistent through morph.

@amanasifkhalid
Copy link
Contributor Author

Would be interested in seeing stats on how many methods are consistent through morph.

Yeah, I plan on adding metrics in the next push. After morph seems like a good place to run profile repair, since that's where we currently try to fill in the blanks with fgComputeBlockWeights, optSetBlockWeights, etc.

@amanasifkhalid amanasifkhalid merged commit aecae2c into dotnet:main Jan 7, 2025
155 of 176 checks passed
@amanasifkhalid amanasifkhalid deleted the profile-checks-tail-merge branch January 7, 2025 17:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants