[PATCH] D84108: [SimplifyCFG][LoopRotate] SimplifyCFG: disable common instruction hoisting by default, enable late in pipeline

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

In D84108#2231809 <https://reviews.llvm.org/D84108#2231809>, @xbolva00 wrote:

>>> loops not rotated due to the header size
>
> Increase that limit? Any disadvantages?

Loops not being rotated is actually a symptom.
While it may be worthwhile to increase that threshold,
that isn't really the fix, because then we still have hoisted the code.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84108