[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