[PATCH] D78314: [mlir][Pass] Add support for generating local crash reproducers
River Riddle via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 16 13:23:31 PDT 2020
rriddle marked 2 inline comments as done.
rriddle added inline comments.
================
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.
----------------
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.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78314/new/
https://reviews.llvm.org/D78314
More information about the llvm-commits
mailing list