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

[Simon Tatham via Phabricator via llvm-commits]

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