-
Notifications
You must be signed in to change notification settings - Fork 570
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
Fix 130 #813
Fix 130 #813
Conversation
so bb can be associated with function in output. only captures BBs that have a rule match, otherwise, there might be too much data captured. closes #130.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add bug fixes, new features, breaking changes and anything else you think is worthwhile mentioning to the master (unreleased)
section of CHANGELOG.md. If no CHANGELOG update is needed add the following to the PR description: [x] No CHANGELOG update needed
CHANGELOG updated or no update needed, thanks! 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wonderful! thank you for all your work again @williballenthin! LGTM 🚀
record the layout of basic blocks within functions to enable the display of the containing function when a BB-scoped rule matches. stores this in
.meta.analysis.layout.functions.0x401000.matched_basic_blocks = [0x401000, 0x401005, ...]
.only captures the basic blocks that have a match. this increases the JSON document size of k32 from 3.4 to 3.6MB; capturing all BBs results in a 3.9MB document.
closes #130
Checklist