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 clang r344100 into 7.0.1 #38869

Closed
DimitryAndric opened this issue Nov 1, 2018 · 4 comments
Closed

Merge clang r344100 into 7.0.1 #38869

DimitryAndric opened this issue Nov 1, 2018 · 4 comments
Assignees
Labels
bugzilla Issues migrated from bugzilla clang Clang issues not falling into any other category

Comments

@DimitryAndric
Copy link
Collaborator

Bugzilla Link 39521
Resolution FIXED
Resolved on Nov 30, 2018 10:15
Version 7.0
OS All
Blocks #38454
CC @emaste,@zygoloid,@tstellar
Fixed by commit(s) r344100 r348012

Extended Description

Please merge https://reviews.llvm.org/rC344100 ("clang: Allow ifunc resolvers to accept arguments") into clang 7.0.1. This allows ifunc resolvers to take arguments.

@DimitryAndric
Copy link
Collaborator Author

assigned to @jyknight

@tstellar
Copy link
Collaborator

Hi James,

Do you think this is safe to merge?

@llvmbot
Copy link
Member

llvmbot commented Nov 21, 2018

FWIW, FreeBSD's build depends on this. It would be nice if we could get it into Linux/MacOS packages sooner rather than later (we're applying it to both our in-tree LLVM and our packages).

@tstellar
Copy link
Collaborator

Merged: r348012

@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 clang Clang issues not falling into any other category
Projects
None yet
Development

No branches or pull requests

3 participants