[PATCH] D44880: [Hexagon] peel loops with runtime small trip counts

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 2 09:21:11 PDT 2018


mkazantsev added a comment.

I think this patch may cause some inefficiencies in terms of compile time and generated code in case if unroller decides to make full unrolling or partial unrolling with user-forced number of iterations, see discussion of https://reviews.llvm.org/D44919. However it is not a correctness issue, i.e. resulting code will still produce right results.

Currently it fails by assert after the logic has changed, to fix it I propose to add `canPeel` check before forcing peel count.


Repository:
  rL LLVM

https://reviews.llvm.org/D44880





More information about the llvm-commits mailing list