[PATCH] D86263: [ELF] Keep st_type for symbol assignment

Peter Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 20 02:42:46 PDT 2020


psmith added a comment.

Not had a chance to look into the changes in detail yet, but on first glance it looks good. Given that the compiler uses STT_FUNC for the entry of a symbol I think it is reasonable to retain the type only for an alias.



================
Comment at: lld/docs/ELF/linker_script.rst:37
+
+The ``st_size`` field is 0.
+
----------------
May be worth using "The ``st_size`` field is set to 0."


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86263



More information about the llvm-commits mailing list