[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