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

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

Just because it's technically possible to put the entire pointer into an index doesn't make it legal, generally (e.g. that's not how GEP works).  But if Intel wants to define its intrinsics that way, fine, I guess.  Please try to verify that some other compiler behaves this way, and add a testcase.

I'm not sure I follow how you would reach useful memory on a 64-bit system; the indices are 32 bits, right?

Do we lower any Intel intrinsics to llvm.masked.gather/scatter?  If we do, we might need to change clang as well.


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

https://reviews.llvm.org/D58805