[PATCH] D69446: [llvm][MachineOutliner] Add support for repeating machine outliner N times.

David Tellenbach via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 1 05:05:50 PDT 2019


tellenbach added inline comments.


================
Comment at: llvm/lib/CodeGen/MachineOutliner.cpp:99
+static cl::opt<unsigned> OutlinerReruns(
+    "outliner-reruns", cl::init(0), cl::Hidden,
+    cl::desc(
----------------
Could this be renamed to something like `machine-outliner-[re]runs`? I find an option `outliner...` to be somewhat ambiguous (there is e.g. outlining during HotColdSpilling) and the prefix `machine` would be more consistent here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69446





More information about the llvm-commits mailing list