Various

substrait-io · Jan 6, 2025 · e345893 · e345893
1 parent 649b195
commit e345893
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/include/substrait-mlir/Dialect/Substrait/IR/CMakeLists.txt b/include/substrait-mlir/Dialect/Substrait/IR/CMakeLists.txt
@@ -1,7 +1,7 @@
 add_mlir_dialect(SubstraitOps substrait)
 add_dependencies(MLIRSubstraitDialect MLIRSubstraitOpsIncGen)
 
-# Add SetOpKind attribute
+# Add Enums
 set(LLVM_TARGET_DEFINITIONS SubstraitOps.td)
 mlir_tablegen(SubstraitEnums.h.inc -gen-enum-decls)
 mlir_tablegen(SubstraitEnums.cpp.inc -gen-enum-defs)

diff --git a/lib/Target/SubstraitPB/Export.cpp b/lib/Target/SubstraitPB/Export.cpp
@@ -159,6 +159,7 @@ SubstraitExporter::exportType(Location loc, mlir::Type mlirType) {
     return std::move(type);
   }
 
+  // Handle tuple types.
   if (auto tupleType = llvm::dyn_cast<TupleType>(mlirType)) {
     auto structType = std::make_unique<proto::Type::Struct>();
     for (mlir::Type fieldType : tupleType.getTypes()) {