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

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 26 09:53:09 PDT 2017


pcc added inline comments.


================
Comment at: unittests/Linker/LinkModulesTest.cpp:939
+}
+
 } // end anonymous namespace
----------------
belleyb wrote:
> 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.
You mean llvm-link, right?


https://reviews.llvm.org/D34450





More information about the llvm-commits mailing list