[PATCH] D43878: [LoopUnroll] Simplify induction variables after peeling too.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 28 11:48:30 PST 2018


efriedma added inline comments.


================
Comment at: test/Transforms/LoopUnroll/peel-loop-conditions.ll:2
 ; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
 ; RUN: opt < %s -S -loop-unroll -verify-dom-info -simplifycfg | FileCheck %s
 
----------------
Putting -simplifycfg is probably more confusing than helpful, in terms of understanding what the pass actually does.


================
Comment at: test/Transforms/LoopUnroll/peel-loop-conditions.ll:47
 
 for.body:
   %i.05 = phi i32 [ 0, %for.body.lr.ph ], [ %inc, %for.inc ]
----------------
We don't peel this loop on master...?  Or is this based on top of D43876?


https://reviews.llvm.org/D43878





More information about the llvm-commits mailing list