[Mlir-commits] [mlir] [mlir][Pass] Enable the option for reproducer generation without crashing (PR #75421)

[Mehdi Amini]

================
@@ -151,6 +151,21 @@ struct MlirOptMainConfigCLOptions : public MlirOptMainConfig {
 
     static cl::list<std::string> passPlugins(
         "load-pass-plugin", cl::desc("Load passes from plugin library"));
+
+    static cl::opt<std::string, /*ExternalStorage=*/true> reproduerFilename(
+        "mlir-reproducer-filename",
+        llvm::cl::desc("Location of the .mlir reproducer output file path"
+                       " the --mlir-generate-reproducer=1"
+                       " (no crash required)"),
+        cl::location(reproduerFilenameFlag), cl::init(""),
+        cl::value_desc("filename"));
+    static cl::opt<bool, /*ExternalStorage=*/true> generateReproducer(
+        "mlir-generate-reproducer",
+        llvm::cl::desc("Generate a reproducer at"
+                       " --mlir-reproducer-filename=<filename> "
+                       " (no crash required)"),
+        cl::location(generateReproducerFlag), cl::init(false));
----------------
joker-eph wrote:

I don't follow what you're asking about here?

You're introducing 2 new options, I'm saying you can introduce one only. But that won't conflict with the `mlir-pass-pipeline-crash-reproducer` option right?

https://github.com/llvm/llvm-project/pull/75421