[PATCH] D113939: [runtime-unroll] Remove restriction about unrolling multiple exit loops
    Roman Lebedev via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Tue Nov 16 11:45:07 PST 2021
    
    
  
lebedev.ri added a comment.
In D113939#3135607 <https://reviews.llvm.org/D113939#3135607>, @nikic wrote:
> 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.
+1 to that, but i think unrolling is generally a good thing that we *should* be doing.
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