[clang] [Clang] Fix isWeakImported() to traverse redeclaration chain for avai… (PR #181482)

via cfe-commits cfe-commits at lists.llvm.org
Wed May 6 19:24:46 PDT 2026


kevinlzh1108 wrote:

> > Hi @vsapsai , sorry for the disturbance.I'm new to LLVM, is there anything else needed before this can be merged? Does it need additional approvals?
> 
> In general you might need to wait more if there are ongoing requests from other reviewers or if the change is controversial. Or if an approver's knowledge in the area isn't sufficient to merge the change.
> 
> In this case I believe my approval is sufficient. It's great you've asked for clarification about the merging norms! Can you please disable "Keep my email addresses private" setting in GitHub profile per https://llvm.org/docs/DeveloperPolicy.html#email-addresses ? After I merge the PR you can enable it again.

  Done. I've disabled "Keep my email addresses private" in my GitHub settings.

https://github.com/llvm/llvm-project/pull/181482


More information about the cfe-commits mailing list