[PATCH] D145208: [COFF] Add MC support for emitting IMAGE_WEAK_EXTERN_ANTI_DEPENDENCY symbols

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 6 13:05:54 PDT 2023


mstorsjo accepted this revision.
mstorsjo added a comment.
This revision is now accepted and ready to land.

In D145208#4192722 <https://reviews.llvm.org/D145208#4192722>, @mstorsjo wrote:

> Looks reasonable to me, but I don't feel entirely confident about what this does in the common MC layer changes.

Ok, if there's nobody else who wants to chime in here, then I guess this is fine, and I trust your judgement on it. Sorry for not following up earlier.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D145208/new/

https://reviews.llvm.org/D145208



More information about the llvm-commits mailing list