[PATCH] D77643: Keep output file after successful execution of mlir-opt

[Jean-Michel Gorius via Phabricator via llvm-commits]

Kayjukh added inline comments.


================
Comment at: mlir/examples/standalone/standalone-opt/standalone-opt.cpp:49
 
+static llvm::cl::opt<bool> allowUnregisteredDialects(
+    "allow-unregistered-dialect",
----------------
LukasSommerTu wrote:
> Kayjukh wrote:
> > Should this be part of a separate patch? It is not really related to your proposed change.
> I can make a separate patch for that, but then we would need to pass a magic constant `true` or `false` to `MlirOptMain`, to match the number of required parameters. 
> 
> But if you prefer to have that in a separate patch, I can extract it. What do you prefer?
I was thinking about making a patch for this command line argument first, and then adding the call to `keep`. At least I would update the title and description to mention these changes.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77643