================ @@ -351,6 +354,16 @@ struct BufferizationOptions { /// used. UnknownTypeConverterFn unknownTypeConverterFn = nullptr; + // Use during type conversion to determine the memory space for memref based + // on the originanl tensor type + GetMemorySpaceFn getMemorySpaceFn = nullptr; ---------------- matthias-springer wrote:
I think we don't need both `defaultMemorySpace` and `getMemorySpaceFn`. Can you rename this field to `defaultMemorySpaceFn` and give it a default lambda of `return Attribute();`? Also clarify in the documentation of the field that this lambda is only used when the memory space cannot be inferred. https://github.com/llvm/llvm-project/pull/78484 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits