[PATCH] D103748: [LoopUnroll] Clamp unroll count to MaxTripCount

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jun 6 16:05:28 PDT 2021


efriedma added inline comments.


================
Comment at: llvm/test/Transforms/LoopUnroll/multiple-exits.ll:6
 
 ; TODO: We should unroll by 10, not 20 here
 define void @test1() {
----------------
Remove TODO?


================
Comment at: llvm/test/Transforms/LoopUnroll/nonlatchcondbr.ll:161
 
 define void @test3(i32* noalias %A, i1 %cond) {
 ; CHECK-LABEL: @test3(
----------------
Is this still testing what it's supposed to?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103748



More information about the llvm-commits mailing list