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

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 22 01:45:34 PST 2020


dmgreen added a comment.

Yep, what Simon said.  For normal non-post/pre inc MVE loads we already get this correct. The load is marked as not having sideeffects as it is inferred from the pattern in the same way as any other instruction would be. But for Pre/Post inc loads, we do not have patterns as they have two outputs, so are lowered via dag2dag.


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

https://reviews.llvm.org/D73117





More information about the llvm-commits mailing list