[PATCH] D113939: [runtime-unroll] Remove restriction about unrolling multiple exit loops
Philip Reames via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 16 11:42:50 PST 2021
reames added inline comments.
================
Comment at: llvm/lib/Transforms/Utils/LoopUnrollRuntime.cpp:422
if (UnrollRuntimeMultiExit.getNumOccurrences())
return UnrollRuntimeMultiExit;
----------------
nikic wrote:
> The UnrollRuntimeMultiExit and UnrollRuntimeOtherExitPredictable options are now dead.
Er, yeah. I thought about removing them, but apparently didn't actually do so. Oops. :)
Once we have some sanity check benchmark scores, I'll update to remove these in the patch.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D113939/new/
https://reviews.llvm.org/D113939
More information about the llvm-commits
mailing list