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

[Fedor Sergeev via Phabricator via llvm-commits]

fedor.sergeev added inline comments.


================
Comment at: lib/Passes/PassBuilder.cpp:1441
+      Result.sinkCommonInsts(Enable);
+    } else if (ParamName.consume_front("bonus-inst-threshold=")) {
+      APInt BonusInstThreshold;
----------------
I believe you do not want to allow "no-" prefix for this option.
Add Enable to condition?


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

https://reviews.llvm.org/D60675