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

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 22 01:22:25 PST 2020


samparker added a comment.

In D73117#1833090 <https://reviews.llvm.org/D73117#1833090>, @samparker wrote:

> Why isn't this handled properly by the scheduling infrastructure instead? This sounds a bit like a hack...


Sorry Dave, I misread the title! My post-coffee question is, why do we attach SideEffects to patterns and not instruction descriptions anyway?


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

https://reviews.llvm.org/D73117





More information about the llvm-commits mailing list