[PATCH] D45536: Do not keep shared symbols to garbage-collected eliminated DSOs.

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 11 15:39:29 PDT 2018


pcc added inline comments.


================
Comment at: lld/ELF/Driver.cpp:1065
+// created from the DSO. Otherwise, they become dangling references
+// that point to a non-exsitent DSO.
+template <class ELFT>
----------------
nit: non-existent


================
Comment at: lld/test/ELF/as-needed-weak.s:11
+
+# CHECK-NOT: libfoo
+
----------------
Won't this pass without the change?

Probably a better way to test this would be to check that foo is relocated properly.


https://reviews.llvm.org/D45536





More information about the llvm-commits mailing list