[PATCH] D60675: [NewPM] Add Option handling for SimplifyCFG

[Serguei Katkov via Phabricator via llvm-commits]

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


================
Comment at: lib/Passes/PassBuilder.cpp:1441
+      Result.sinkCommonInsts(Enable);
+    } else if (ParamName.consume_front("bonus-inst-threshold=")) {
+      APInt BonusInstThreshold;
----------------
fedor.sergeev wrote:
> I believe you do not want to allow "no-" prefix for this option.
> Add Enable to condition?
Make sense. Will fix that before landing if there is no other objections.


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

https://reviews.llvm.org/D60675