[Mlir-commits] [mlir] [mlir][python] generate value builders (PR #68308)

[Jacques Pienaar]

================
@@ -982,11 +1032,12 @@ static void emitOpBindings(const Operator &op, raw_ostream &os) {
   }
 
   emitRegionAttributes(op, os);
-  emitDefaultOpBuilder(op, os);
+  llvm::SmallVector<std::string> functionArgs = emitDefaultOpBuilder(op, os);
   emitOperandAccessors(op, os);
   emitAttributeAccessors(op, os);
   emitResultAccessors(op, os);
   emitRegionAccessors(op, os);
+  emitValueBuilder(op, functionArgs, os);
----------------
jpienaar wrote:

Any reason to not just invoke the value builder emitter inside the op builder one?

https://github.com/llvm/llvm-project/pull/68308