[PATCH] D138456: Add new option choices for print-changed that do not output llc changes.

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 21 13:47:48 PST 2022


aeubanks added a comment.

if clang is giving you too much, you can always not run the backend passes with `-emit-llvm` when debugging



================
Comment at: llvm/include/llvm/IR/PrintPasses.h:37
+  OptDotCfgVerbose,
+  OptDotCfgQuiet
 };
----------------
MaskRay wrote:
> The number of modes is massive. Shall we use another option to indicate the intent (only show non-codegen passes)?
agreed, this is growing exponentially because it really should be multiple flags

something like
`print-changed`
`print-changed-diff[=color]`
`print-ir-pass-changed` (default true)
`print-mir-pass-changed` (default true)
`print-dotcfg`
`print-changed-verbose` (affects all change printers)
...

this will allow users to specify more than one change printer at once which I think is fine (could manually error out if not desired, but I don't think that's necessary)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138456



More information about the llvm-commits mailing list