[PATCH] D72537: [NewPM] Port MergeFunctions pass

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 14 11:32:43 PST 2020


nikic marked an inline comment as done.
nikic added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/MergeFunctions.cpp:327
+                                          ModuleAnalysisManager &AM) {
+  MergeFunctions MF;
+  if (!MF.runOnModule(M))
----------------
leonardchan wrote:
> I don't suppose you know off the top of your head if not doing something like `skipModule` will have any impact to build times?
I believe `skipModule()` in legacy PM is only used for the opt-bisect functionality, so this shouldn't impact build times.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72537





More information about the llvm-commits mailing list