-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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 tvm::support::hexdump() debug utility #6154
Conversation
* also adjust compiler flags.
src/support/hexdump.h
Outdated
namespace tvm { | ||
namespace support { | ||
|
||
void hexdump(const std::string& s, std::ostream& os); |
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.
HexDump as per Google C convention
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.
done
src/support/hexdump.h
Outdated
namespace tvm { | ||
namespace support { | ||
|
||
void hexdump(const std::string& s, std::ostream& os); |
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.
document via doxygen
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.
done
Thanks @areusch ! |
Adding a debug utility which can be useful to inspect binary data in the C++ runtime.
As part of this change, build C++ tests against a statically-linked TVM runtime so that
-fvisibility=hidden
can still be used but doesn't apply to unit tests.