[PATCH] D128505: [AArch64] Update SME load/store intrinsics to work on opaque pointers.

[Sander de Smalen via Phabricator via llvm-commits]

sdesmalen added inline comments.


================
Comment at: llvm/include/llvm/IR/IntrinsicsAArch64.td:2589-2603
   class SME_Load_Store_B_Intrinsic
     : DefaultAttrsIntrinsic<[],
         [llvm_nxv16i1_ty, llvm_ptr_ty, llvm_i64_ty, llvm_i32_ty], []>;
   class SME_Load_Store_H_Intrinsic
     : DefaultAttrsIntrinsic<[],
-        [llvm_nxv16i1_ty, LLVMPointerType<llvm_i16_ty>, llvm_i64_ty, llvm_i32_ty], []>;
+        [llvm_nxv16i1_ty, llvm_ptr_ty, llvm_i64_ty, llvm_i32_ty], []>;
   class SME_Load_Store_S_Intrinsic
----------------
c-rhodes wrote:
> a single `SME_Load_Store_Intrinsic` class will do now since they're all the same?
That's true, but that's likely to change in the near future because the we shouldn't be using a nxv16i1 predicate type for all these intrinsics.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D128505/new/

https://reviews.llvm.org/D128505