[PATCH] D102928: [unroll] Use SCEV to evaluate branch conditions in cost model

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun May 23 20:27:03 PDT 2021


mkazantsev added inline comments.


================
Comment at: llvm/test/Transforms/LoopUnroll/unroll-cost-symbolic-execute.ll:671
+; CHECK:       backedge.8:
+; CHECK-NEXT:    ret i32 0
 ;
----------------
mkazantsev wrote:
> I like the change, but the results like this scare me a lot. Is this really what we want to see in code?
Ah, it's dead code... Nvm then.


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

https://reviews.llvm.org/D102928



More information about the llvm-commits mailing list