[PATCH] D58805: [X86] Remove IntrArgMemOnly from target specific gather/scatter intrinsics

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 28 17:57:01 PST 2019


craig.topper added a comment.

There are intrinsics with 32 bit indices and intrinsics with 64-bit indices. The data elements are 32 bits or 64 bits integer or floating. And we support basically all permutations of the data and indices. On 64-bit targets the 32-bit indices are sign extended. And 32-bit targets the 64-bit indices are truncated I think.

We don't convert any target specific intrinsics to masked.gather/scatter in clang. It would require combining the 3 disected address pieces into gep or ptrtoint/inttoptr+arithmetic so I haven't looked into it.


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

https://reviews.llvm.org/D58805





More information about the llvm-commits mailing list