[Mlir-commits] [mlir] [tblgen] Use `emitError` for inferResultTypes failures (PR #165488)

[Stef Lindall]

================
@@ -2681,7 +2681,7 @@ void OpEmitter::genSeparateArgParamBuilder() {
                       {1}.regions, inferredReturnTypes)))
           {1}.addTypes(inferredReturnTypes);
         else
-          ::mlir::detail::reportFatalInferReturnTypesError({1});
+          ::mlir::detail::emitInferReturnTypesError({1});
----------------
bethebunny wrote:

Gotcha. What is your recommendation here? Generally I would gravitate towards patterns like `FailureOr` but as you say the builder API doesn't have a clear place to put this.

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