[PATCH] D92589: Introduce new option -print-changed-only and associated change printer.

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 9 21:24:16 PST 2020


aeubanks added a comment.

I like that idea. Actually I think you might even be able to fit it all in one flag. Just passing `-print-changed` gives you the default, and you can say something like `-print-changed=quiet` to not print on skipped passes (as an example). And apparently this is possible with `cl::opt`, e.g. I found

  static cl::opt<RunOutliner> EnableMachineOutliner(
      "enable-machine-outliner", cl::desc("Enable the machine outliner"),
      cl::Hidden, cl::ValueOptional, cl::init(TargetDefault),
      cl::values(clEnumValN(AlwaysOutline, "always",
                            "Run on all functions guaranteed to be beneficial"),
                 clEnumValN(NeverOutline, "never", "Disable all outlining"),
                 // Sentinel value for unspecified option.
                 clEnumValN(AlwaysOutline, "", "")));


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92589



More information about the llvm-commits mailing list