[MLIR] Improve debug messages in BuiltinTypes
authorGeoffrey Martin-Noble <gcmn@google.com>
Fri, 6 Aug 2021 22:59:50 +0000 (15:59 -0700)
committerGeoffrey Martin-Noble <gcmn@google.com>
Wed, 6 Oct 2021 16:42:13 +0000 (09:42 -0700)
It's nice for users to have more information when debugging failures and
these are only triggered in a failure path.

Reviewed By: mehdi_amini

Differential Revision: https://reviews.llvm.org/D107676

mlir/lib/IR/BuiltinTypes.cpp

index 3589ed1..a87dd55 100644 (file)
@@ -445,10 +445,14 @@ LogicalResult VectorType::verify(function_ref<InFlightDiagnostic()> emitError,
     return emitError() << "vector types must have at least one dimension";
 
   if (!isValidElementType(elementType))
-    return emitError() << "vector elements must be int/index/float type";
+    return emitError()
+           << "vector elements must be int/index/float type but got "
+           << elementType;
 
   if (any_of(shape, [](int64_t i) { return i <= 0; }))
-    return emitError() << "vector types must have positive constant sizes";
+    return emitError()
+           << "vector types must have positive constant sizes but got "
+           << shape;
 
   return success();
 }