-
Notifications
You must be signed in to change notification settings - Fork 478
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
Link bytecode using references #312
base: master
Are you sure you want to change the base?
Conversation
b9f2779
to
be228e6
Compare
9232d1a
to
8dbd9a4
Compare
8dbd9a4
to
8cc1dee
Compare
d8c2553
to
01dd7ca
Compare
@r0qs I see there's a merge commit. Can you rebase it on top of |
01dd7ca
to
7536d10
Compare
Yes. I did it now, but as the commits were very old and the file structures changed significantly, the OP changes needed to be discarded during the rebase. However, I added the OP as a co-author of my changes, since I reused part of his solution. |
Co-authored-by: arthcp <arth.c.patel@gmail.com>
7536d10
to
954e6f9
Compare
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
Support to provide
linkReferences
as optional third parameter forlinkBytecode
. If it is not provided thenfindLinkReferences
is used to find it. If library name is not present inlinkReferences
then it falls back to manual search.Fixes #193