[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:

I just looked into this, but this wouldn't be universally applicable in this file: some of the cases are streaming multiple piece together to form the error message, and some function that have this pattern are not returning a Value. 

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