[PATCH] D138283: [SVE] Enhanced loop access analysis

dewen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 18 23:28:45 PST 2022


dewen added a comment.

In D138283#3937896 <https://reviews.llvm.org/D138283#3937896>, @efriedma wrote:

> Please simplify your testcase so it only contains the relevant loop, and doesn't contain any other irrelevant instructions.
>
> I don't understand the relationship between "store that stores a constant value" and "loop has no dependencies"; as far as I can tell, they're unrelated.  Even for your testcase, the load can depend on the store.

Thank you. I'll simplify the testcases before submitting them.
Thank you again for your review


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

https://reviews.llvm.org/D138283



More information about the llvm-commits mailing list