Skip to content

Improve Stack Names For Dynamically Generated JIT Helpers - #2455

Merged
Brian Robbins (brianrob) merged 2 commits into
microsoft:mainfrom
brianrob:brianrob/jithelpers
Aug 18, 2026
Merged

Brian Robbins (brianrob) merged 2 commits into
microsoft:mainfrom
brianrob:brianrob/jithelpers

Conversation

@brianrob

@brianrob Brian Robbins (brianrob) commented Aug 17, 2026 •

Copy link
Copy Markdown
Member

Updates ETLX conversion to recognize JitHelper method events from both live and rundown providers.

JIT helpers are associated with a process-specific synthetic module and displayed as:

generatedruntimehelpers!<helper-name>

For example:

generatedruntimehelpers!WriteBarrier
generatedruntimehelpers!VSD_ResolveStub

Non-helper method resolution remains unchanged.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@brianrob
Brian Robbins (brianrob) requested a review from a team as a code owner August 17, 2026 20:45
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

// TODO FIX NOW remove the jittedMethods ones.
private List<MethodLoadUnloadVerboseTraceData> jittedMethods;
// TODO FIX NOW remove the methodLoadEvents ones.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Is this TODO FIX NOW still relevant? If so, maybe a more in-depth comment about what's going here would be helpful.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I don't think this is relevant anymore, but I need to check. There are a lot of these, and I've been thinking about doing a pass over them to clean things up. I'll plan to handle this as part of that pass.

@brianrob
Brian Robbins (brianrob) merged commit 2ba7128 into microsoft:main Aug 18, 2026
5 checks passed
@brianrob
Brian Robbins (brianrob) deleted the brianrob/jithelpers branch August 18, 2026 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants