[PATCH] D73117: [ARM] Mark MVE loads/store as not having side effects

Simon Tatham via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 22 01:50:25 PST 2020


simon_tatham added a comment.

Anyway: should this not also apply to writeback gather/scatter instructions, like `MVE_VLDRWU32_qi_pre`? Those also have two outputs, so they're selected by C++ code, and as far as I can see their `hasSideEffects` flags are still stuck on `?` at the moment.


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

https://reviews.llvm.org/D73117





More information about the llvm-commits mailing list