[PATCH] D16120: [ThinLTO] Remove imported available externally defs from comdats.

[Mehdi AMINI via llvm-commits]

joker.eph added inline comments.

================
Comment at: lib/Linker/LinkModules.cpp:733
@@ +732,3 @@
+  // Remove functions imported as available externally defs from comdats,
+  // as this is a declaration for the linker, and will be dropped eventually.
+  auto *GO = dyn_cast_or_null<GlobalObject>(&GV);
----------------
I don't understand why we need to this?

================
Comment at: lib/Linker/LinkModules.cpp:737
@@ +736,3 @@
+    // The IRMover should not have placed any imported declarations in
+    // a comdat.
+    assert(GO->hasAvailableExternallyLinkage() &&
----------------
This comment looks like it is a bug in the IRMover that you're working around here?


http://reviews.llvm.org/D16120