fixup! [mlir][vector] Add convenience types for scalable vectors

Inline implementation, add comment.

Author: banach-space

mlir/include/mlir/IR/VectorTypes.h

@@ -16,17 +16,29 @@
 namespace mlir {
 namespace vector {
 
+/// A vector type containing at least one scalable dimension
 class ScalableVectorType : public VectorType {
 public:
   using VectorType::VectorType;
 
-  static bool classof(Type type);
+  static bool classof(Type type) {
+    auto vecTy = llvm::dyn_cast<VectorType>(type);
+    if (!vecTy)
+      return false;
+    return vecTy.isScalable();
+  }
 };
 
+/// A vector type with no scalable dimensions
 class FixedWidthVectorType : public VectorType {
 public:
   using VectorType::VectorType;
-  static bool classof(Type type);
+  static bool classof(Type type) {
+    auto vecTy = llvm::dyn_cast<VectorType>(type);
+    if (!vecTy)
+      return false;
+    return !vecTy.isScalable();
+  }
 };
 
 } // namespace vector

mlir/lib/IR/CMakeLists.txt

@@ -40,7 +40,6 @@ add_mlir_library(MLIRIR
   Unit.cpp
   Value.cpp
   ValueRange.cpp
-  VectorTypes.cpp
   Verifier.cpp
   Visitors.cpp
   ${pdl_src}