-
Notifications
You must be signed in to change notification settings - Fork 399
[PROF-12743] Runtime stack collection callback registration #4984
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
Open
gyuheon0h
wants to merge
21
commits into
master
Choose a base branch
from
gyuheon0h/prof-12743-runtime-stack-callback
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
61e5744
First commit; declare libdatadog ext api
gyuheon0h 750eeba
Dummy frame emitted
gyuheon0h 1bc1b24
Include internal structures (at least try to))
gyuheon0h bf89a31
Can walk the stack, but no func names yet
gyuheon0h a18d45d
Add safety checks when walking stack
gyuheon0h 1072e0b
Get C frames, and also add some AI generated sanity check tests
gyuheon0h acaa374
Clean up tests
gyuheon0h 1139b01
Use all safety checks
gyuheon0h 9a4fe49
Category specific runtime callback
gyuheon0h 4ef334e
Move runtime stuff to own file
gyuheon0h f30b5ee
Try to fix issues across diff ruby versions
gyuheon0h 16db3f4
remove early continue
gyuheon0h c9c7bfc
3.3 issues
gyuheon0h 1a73d22
Fallback to common headers is MJIT headers fail?
gyuheon0h 6988d5a
Move into own extension
gyuheon0h aa6c59d
Move into prof extension
gyuheon0h 05f9f4b
Clean up
gyuheon0h 6c25fde
Gate behind flag
gyuheon0h ef62258
Respond to code cleanliness comments; still need to fix logic/impl
gyuheon0h e985b76
No need to do minimal utf8 checking
gyuheon0h 6e6e165
Register callback before starting CT
gyuheon0h File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.