[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