[PATCH] D87309: [IRSim][IROutliner] Adding option to enable outlining from linkonceodr functions

Andrew Litteken via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 15 20:02:08 PDT 2020


AndrewLitteken added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/IROutliner.cpp:36
+// this is off by default. It should, however, be the default behavior in
+// LTO.
+static cl::opt<bool> EnableLinkOnceODRIROutlining(
----------------
jroelofs wrote:
> The comment seems to suggest that this ought to be set in the LTO pass pipeline, but that doesn't happen in this patch. Is this just for testing?
For now, it is for testing, and we do hope to put the pass in LTO eventually.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D87309/new/

https://reviews.llvm.org/D87309



More information about the llvm-commits mailing list