[PATCH] D70592: [ARM,MVE] Rename and clean up VCTP IR intrinsics.

Simon Tatham via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 2 06:59:04 PST 2019


simon_tatham marked an inline comment as done.
simon_tatham added inline comments.


================
Comment at: llvm/include/llvm/IR/IntrinsicsARM.td:784
+// vctp64 takes v4i1, to work around v2i1 not being a legal MVE type
+def int_arm_mve_vctp64 : Intrinsic<[llvm_v4i1_ty], [llvm_i32_ty], [IntrNoMem]>;
 
----------------
samparker wrote:
> I think we should just remove this definition for now, unless you've already got some clang tests that generate the IR?
I have – those are in the next patch in the stack, D70485. I'm about to add the `vctp64q` ACLE intrinsic which will expand to that IR intrinsic.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70592





More information about the llvm-commits mailing list