[PATCH] D80138: [mlir] ensureRegionTerminator: take OpBuilder

[Nicolas Vasilache via Phabricator via llvm-commits]

nicolasvasilache accepted this revision.
nicolasvasilache added inline comments.


================
Comment at: mlir/include/mlir/IR/OpDefinition.h:1123
+    /// OpBuilder is available at the call site, e.g., in the parser.
     static void ensureTerminator(Region &region, Builder &builder,
                                  Location loc) {
----------------
would it be better to make it explicit by having a small wrapper class and force users to cast?
Making the non-safe use case hard here and everywhere else would have likely saved many hours of engineering,  


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80138