[PATCH] D34450: bug33527 - Linker::LinkOnlyNeeded should import AppendingLinkage globals (release_40)

Benoit Belley via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 26 09:49:38 PDT 2017


belleyb added inline comments.


================
Comment at: unittests/Linker/LinkModulesTest.cpp:939
+}
+
 } // end anonymous namespace
----------------
mehdi_amini wrote:
> Can this be tested with llvm-lto instead of a gtest? That's the preferred way of testing.
No problem. I'll do that.


https://reviews.llvm.org/D34450





More information about the llvm-commits mailing list