[PATCH] D72050: [opt] refactor opt main function

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added inline comments.


================
Comment at: llvm/tools/opt/opt.cpp:693
   if (PassPipeline.getNumOccurrences() > 0) {
-    OutputKind OK = OK_NoOutput;
-    if (!NoOutput)
----------------
`GetOutputKind` is only called once. The change does not seem to improve readability.


================
Comment at: llvm/tools/opt/opt.cpp:944
+    case OK_NoOutput:
+      assert(false);
+      break;
----------------
llvm_unreachable


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

https://reviews.llvm.org/D72050