[PATCH] D106025: [RISCV] Use unordered indexed loads for MGATHER.

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

In D106025#2879250 <https://reviews.llvm.org/D106025#2879250>, @frasercrmck wrote:

> I'm glad you brought this up. I remember having my doubts when first doing the lowering for `MGATHER`. I can't remember if we had a discussion around that or not but it was on my mind. I think at some point in development I went with the conservative option and it wasn't caught in review. I suspect it's okay to use unordered loads. Do you know if other targets do?

X86 documentation for gather says "The values may be read from memory in any order. Memory ordering with other instructions follows the Intel64 memory-ordering model."

ARM's documentation makes no mention of ordering that I can find.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106025