-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add binary file output support #2
Conversation
Port of embedded-gcov[1] for Unikraft. [1] https://github.com/nasa-jpl/embedded-gcov Signed-off-by: Michalis Pappas <[email protected]>
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.
Nice! 😎 I have left inline comments on some minor issues.
One more thing that could be imporoved is the structure of the README
with a brief summary of the process before elaborating on the individual output types, ie something along the lines of:
- Patch code to add
__gcov_exit()
- Select output type and build
- Run script (a brief description of what the script does and sample output can be added here)
Also I noticed some trailing whitespace here and there. You can spot them with git show
.
- Improved README - Added 'used' attribute in gcov patch Signed-off-by: Eduard-Florin Mihailescu <[email protected]>
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.
@Starnox thanks for updating this PR. Besides the rest of the comments, please:
- Rebase on top of Introduce lib-embedded-gcov #1. If we can get it merged before the next iteration of this PR, even better (you are a reviewer and @razvand is the approver).
- Squash 8f0bc5f into d6e5d23.
Thanks!
edfcaf8
to
2c3a9b6
Compare
- Added configuration options to accommodate the feature. - Added patch for the embedded-gcov library to resolve division bug - Included helper script for processing extracted information - Modified README Signed-off-by: Eduard-Florin Mihailescu <[email protected]>
2c3a9b6
to
9ea34c0
Compare
Support for binary file coverage information, based on #1
embedded-gcov
library that fixes a minor bug in the way divisions were madeREADME.md