-
Notifications
You must be signed in to change notification settings - Fork 10.8k
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
[TextAPI] Add DylibReader #75006
[TextAPI] Add DylibReader #75006
Conversation
Add support for reading binary Mach-o dynamic libraries. It uses libObject APIs for extracting information relavant to TAPI and tbd files. This includes but is not limited to load commands encode data like install names, current/compat versions and symbols.
@@ -103,6 +103,10 @@ class GlobalRecord : public Record { | |||
|
|||
bool isFunction() const { return GV == Kind::Function; } | |||
bool isVariable() const { return GV == Kind::Variable; } | |||
void setKind(const Kind &V) { |
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.
In this case I would prefer to pass the argument by value. It is only an uint8_t
enum. There is no need to pass it by reference.
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.
While I agree, I also wanted to keep the const
mostly to ensure that the passed value isn't modified, just assigned. But that puts us back to #75006 (comment)
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.
LGTM
Add support for reading binary Mach-o dynamic libraries. It uses libObject APIs for extracting information relavant to TAPI and tbd files. This includes but is not limited to load commands encode data like install names, current/compat versions and symbols. (cherry picked from commit 634fedd)
…ylibReader (llvm#75006)" This reverts commit aa217eb. This reverts commit 634fedd. This breaks buildbots by introducing cycle dependency between libObject and TextAPI and breaks gcc compiles on buildbots. (cherry picked from commit 1fef0fa)
Add support for reading binary Mach-o dynamic libraries. It uses libObject APIs for extracting information relavant to TAPI and tbd files. This includes but is not limited to load commands encode data like install names, current/compat versions and symbols. (cherry picked from commit 634fedd)
…ylibReader (llvm#75006)" This reverts commit aa217eb. This reverts commit 634fedd. This breaks buildbots by introducing cycle dependency between libObject and TextAPI and breaks gcc compiles on buildbots. (cherry picked from commit 1fef0fa)
Add support for reading binary Mach-o dynamic libraries. It uses libObject APIs for extracting information relavant to TAPI and tbd files. This includes but is not limited to load commands encode data like install names, current/compat versions and symbols.