[PATCH] D130276: [JITLink][COFF] Implement include/alternatename linker directive.

Lang Hames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 25 16:34:01 PDT 2022


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

LGTM. This is really cool -- we've never had a proper way to handle COFF exports!

Speaking of which -- in the long term we may want to move the COFF options parser to libObject -- it would allow us to set the SF_Export flag correctly in that library, and JITLink could re-use the parser from there (since JITLink depends on libObject).


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

https://reviews.llvm.org/D130276



More information about the llvm-commits mailing list