[llvm] [IR] Initial introduction of memset_pattern (PR #97583)

Alex Bradbury via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 8 07:30:43 PST 2024


================
@@ -1219,14 +1220,16 @@ class MemIntrinsic : public MemIntrinsicBase<MemIntrinsic> {
   }
 };
 
-/// This class wraps the llvm.memset and llvm.memset.inline intrinsics.
+/// This class wraps the llvm.memset, llvm.memset.inline, and
+/// llvm.memset.pattern intrinsics.
 class MemSetInst : public MemSetBase<MemIntrinsic> {
 public:
   // Methods for support type inquiry through isa, cast, and dyn_cast:
   static bool classof(const IntrinsicInst *I) {
     switch (I->getIntrinsicID()) {
     case Intrinsic::memset:
     case Intrinsic::memset_inline:
+    case Intrinsic::memset_pattern:
----------------
asb wrote:

I've updated so MemSetPatternInst is no longer in the MemSetInst hierarchy. I do use MemSetBase, but that's not accessed externally in the LLVM codebase right now.

This is a good call, looking more closely I see e.g. some AnyMemSetInst references in the codebase I'd be wanting to audit more closely. Thanks.

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


More information about the llvm-commits mailing list