[Mlir-commits] [mlir] [mlir][ODS] Allow inferring operand types from multiple variables (PR #127517)

[River Riddle]

================
@@ -457,8 +459,10 @@ void Operator::populateTypeInferenceInfo(
       // If the type of the result has already been inferred, do nothing.
       if (infer.inferred)
         continue;
-      int sourceIndex =
-          argumentsAndResultsIndex.lookup(def.getValueAsString("lhs"));
+      std::vector<StringRef> args = def.getValueAsListOfStrings("args");
+      assert(args.size() == 1 &&
+             "multiple arguments for result inference not yet supported.");
----------------
River707 wrote:

Tablegen error reporting is quite awful, but the best that can be done here is likely `PrintFatalError`. Look at other uses in this file for that, there are some TableGen utilities that do this in a uniform way.

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