[PATCH] D49366: Add support for /FUNCTIONPADMIN command-line option

Alexandre Ganea via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 4 09:25:12 PST 2018


aganea planned changes to this revision.
aganea added a comment.

Just for the record, this is not as simple as I initially thought. We should only be padding functions that originate from hotpatch-enabled OBJs. Currently, LLVM currently does not generate /hotpatch <https://docs.microsoft.com/en-us/cpp/build/reference/hotpatch-create-hotpatchable-image?view=vs-2017> OBJs (at least the flag isn't set). In theory, x64 and ARM OBJs should be fine to have this flag enabled. I'll do more research and get back.


Repository:
  rLLD LLVM Linker

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

https://reviews.llvm.org/D49366





More information about the llvm-commits mailing list