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

[Fraser Cormack via Phabricator via llvm-commits]

frasercrmck added a comment.

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?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106025