[PATCH] D113939: [runtime-unroll] Remove restriction about unrolling multiple exit loops

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

This looks reasonable to me on a conceptual level -- we have recently removed multi-exit restrictions for non-runtime unrolling as well.

Of course, this does have the potential to further exacerbate our problem with way too aggressive unrolling.



================
Comment at: llvm/lib/Transforms/Utils/LoopUnrollRuntime.cpp:422
   if (UnrollRuntimeMultiExit.getNumOccurrences())
     return UnrollRuntimeMultiExit;
 
----------------
The UnrollRuntimeMultiExit and UnrollRuntimeOtherExitPredictable options are now dead.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D113939/new/

https://reviews.llvm.org/D113939