Skip to content
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

Merge r343668 into the 7.0 branch : [ELF] - Do not forget to include to .dynsym symbols that were converted to Defined #38601

Closed
orivej opened this issue Oct 11, 2018 · 6 comments
Assignees
Labels
bugzilla Issues migrated from bugzilla lld:ELF

Comments

@orivej
Copy link
Contributor

orivej commented Oct 11, 2018

Bugzilla Link 39253
Resolution FIXED
Resolved on Oct 26, 2018 09:29
Version unspecified
OS Linux
Blocks #38454
CC @rui314,@tstellar
Fixed by commit(s) r343668 r344925
@orivej
Copy link
Contributor Author

orivej commented Oct 11, 2018

assigned to @rui314

@tstellar
Copy link
Collaborator

Hi Rui,

Is this OK to merge?

https://reviews.llvm.org/rL343668

@rui314
Copy link
Member

rui314 commented Oct 22, 2018

Yes.

@tstellar
Copy link
Collaborator

Merged: r344925

@tstellar
Copy link
Collaborator

*** Bug llvm/llvm-bugzilla-archive#39450 has been marked as a duplicate of this bug. ***

@tstellar
Copy link
Collaborator

mentioned in issue llvm/llvm-bugzilla-archive#39450

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 10, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugzilla Issues migrated from bugzilla lld:ELF
Projects
None yet
Development

No branches or pull requests

3 participants