From e3458935562892ad208b45ecca063e80ea46b2d8 Mon Sep 17 00:00:00 2001 From: Dalia Shaaban <144673861+dshaaban01@users.noreply.github.com> Date: Fri, 27 Dec 2024 19:01:27 +0100 Subject: [PATCH] Various --- include/substrait-mlir/Dialect/Substrait/IR/CMakeLists.txt | 2 +- lib/Target/SubstraitPB/Export.cpp | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/include/substrait-mlir/Dialect/Substrait/IR/CMakeLists.txt b/include/substrait-mlir/Dialect/Substrait/IR/CMakeLists.txt index 3d8b935..ec15e96 100644 --- 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 index dc05cb2..1b8bff0 100644 --- 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(mlirType)) { auto structType = std::make_unique(); for (mlir::Type fieldType : tupleType.getTypes()) {