[PATCH] D86988: [Inliner] Run always-inliner in inliner-wrapper

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 24 23:31:54 PDT 2020


aeubanks added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/Inliner.cpp:98
+static cl::opt<bool> DisableAlwaysInlinerInModuleWrapper(
+    "disable-always-inliner-in-module-wrapper", cl::init(false), cl::Hidden);
+
----------------
davidxl wrote:
> this should be true by default for now.
Is there any reason to delay flipping this? Else it won't have any effect being behind a flag since nobody will turn it on. Unless you'd like the flip to be in a separate change?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86988



More information about the llvm-commits mailing list