[PATCH] D32544: LTO: Mark undefined module inline asm symbols as visible outside of ThinLTO.

[Davide Italiano via Phabricator via llvm-commits]

davide added inline comments.


================
Comment at: llvm/lib/Object/IRSymtab.cpp:166-168
+    // Undefined module asm symbols act as GC roots and are implicitly used.
+    if (Flags & object::BasicSymbolRef::SF_Undefined)
+      Sym.Flags |= 1 << storage::Symbol::FB_used;
----------------
With this patch in place, do we still need the call to `collectAsmUndefined` in the ThinLTO backend? (I understand we need it for LTO, still)


https://reviews.llvm.org/D32544