[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