[PATCH] D83275: [llc] (almost) remove `--print-machineinstrs`

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/lib/CodeGen/TargetPassConfig.cpp:144
 
-static cl::opt<std::string> PrintMachineInstrs(
-    "print-machineinstrs", cl::ValueOptional, cl::desc("Print machine instrs"),
-    cl::value_desc("pass-name"), cl::init("option-unspecified"), cl::Hidden);
+// FIXME: remove this after switching to NPM.
+static cl::opt<bool>
----------------
Or GlobalISel, whichever gets there first...


================
Comment at: llvm/test/CodeGen/ARM/taildup-branch-weight.ll:1
-; RUN: llc -mtriple=arm-eabi -print-machineinstrs=tailduplication -tail-dup-size=100 \
+; RUN: llc -mtriple=arm-eabi -print-after=tailduplication -tail-dup-size=100 \
 ; RUN:      -enable-tail-merge=false -disable-cgp %s -o /dev/null 2>&1 \
----------------
Most of these still look like they can easily be replaced with stop-after


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83275