[PATCH] D78314: [mlir][Pass] Add support for generating local crash reproducers

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini accepted this revision.
mehdi_amini added inline comments.
This revision is now accepted and ready to land.


================
Comment at: mlir/docs/PassManagement.md:1001
+`pass-pipeline-minimal-reproducer` on the command line, that signals that the
+pass manager should attempt to generate a "minimal" reproducer. This will
+attempt to generate a reproducer containing IR right before the pass that fails.
----------------
rriddle wrote:
> jpienaar wrote:
> > minimal has to me a size connotation (e.g., "minimum amount of code needed to make the bug appear when executed."), and here it is more about scope than size (e.g., you only need to run one pass, but your reproducer may be huge). Would "local" or "pass-pipeline-dump-before-first-fail" work? Alternatively, where do you see someone not wanting this behavior?
> I'm fine with local.
> 
> This type of behavior isn't desirable if there are multi-pass interactions. It isn't guaranteed that the problem is with the pass that it is failing.
Interestingly `local` was confusing to me since I thought of it as opposite to "remote" ;)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78314