[Mlir-commits] [mlir] [mlir][llvm] Add support for memset.inline (PR #115711)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Tue Nov 12 03:22:14 PST 2024


================
@@ -931,38 +939,52 @@ static bool areAllIndicesI32(const DestructurableMemorySlot &slot) {
 }
 
 //===----------------------------------------------------------------------===//
-// Interfaces for memset
+// Interfaces for memset && memset.inline
 //===----------------------------------------------------------------------===//
 
-bool LLVM::MemsetOp::loadsFrom(const MemorySlot &slot) { return false; }
+template <class MemsetLike>
+static bool memsetCanRewire(MemsetLike op, const DestructurableMemorySlot &slot,
----------------
PikachuHyA wrote:

Thank you for your suggestion. I want to point out that the naming convention of the `memsetXXX` functions is consistent with the `memcpyXXX` functions in the same file. If we need to adopt the new naming convention, I am more than happy to make the changes. Please confirm whether the new naming style should be applied.


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


More information about the Mlir-commits mailing list