[mlir][ods] Custom builder with no params

Incorrect generation of custom build method without any params.

diff --git a/mlir/tools/mlir-tblgen/OpDefinitionsGen.cpp b/mlir/tools/mlir-tblgen/OpDefinitionsGen.cpp
index 58a1b33..f89547d 100644 (file)
--- a/mlir/tools/mlir-tblgen/OpDefinitionsGen.cpp
+++ b/mlir/tools/mlir-tblgen/OpDefinitionsGen.cpp
@@ -1191,8 +1191,9 @@ void OpEmitter::genBuilder() {
         std::string paramStr =
             skipParamGen ? params.str()
                          : llvm::formatv("::mlir::OpBuilder &{0}, "
-                                         "::mlir::OperationState &{1}, {2}",
-                                         builder, builderOpState, params)
+                                         "::mlir::OperationState &{1}{2}{3}",
+                                         builder, builderOpState,
+                                         params.empty() ? "" : ", ", params)
                                .str();
         auto *method =
             opClass.addMethodAndPrune("void", "build", properties, paramStr);