[Mlir-commits] [mlir] [MLIR][Linalg] Harden parsing Linalg named ops (PR #145337)

[Mehdi Amini]

================
@@ -500,8 +531,13 @@ class RegionBuilderHelper {
         return builder.create<complex::SubOp>(arg0.getLoc(), arg0, arg1);
       if (allFloatingPoint)
         return builder.create<arith::SubFOp>(arg0.getLoc(), arg0, arg1);
-      if (allBool)
+      if (allBool) {
+        if (emitError) {
----------------
joker-eph wrote:

There is a return inside a if statement, how would you simplify this? I could see it with a macro maybe but I'm not sure how to do this with a function.
(the error message is also slightly different between all the cases below)

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