[PATCH] D34619: [ARM] Enable partial and runtime unrolling

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 26 07:52:02 PDT 2017


SjoerdMeijer added a comment.

Hi Sam, 
Where are the tests? ;-) The ad hoc way of calculating the trip count (or the variable) in ##getTripCountValue()## doesn't feel entirely right, is this not what SCEV should be providing?
Cheers,
Sjoerd.


https://reviews.llvm.org/D34619





More information about the llvm-commits mailing list