[PATCH] D142255: [WIP] Loop peeling opportunity for identity operators

Jamie Schmeiser via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 24 11:17:36 PST 2023


jamieschmeiser added inline comments.


================
Comment at: llvm/test/Transforms/LoopUnroll/AArch64/runtime-loop.ll:27
   %indvars.iv = phi i64 [ %indvars.iv.next, %for.body ], [ 0, %entry ]
-  %sum.02 = phi i32 [ %add, %for.body ], [ 0, %entry ]
+  %sum.02 = phi i32 [ %add, %for.body ], [ 1, %entry ]
   %arrayidx = getelementptr inbounds i32, ptr %a, i64 %indvars.iv
----------------
mkazantsev wrote:
> Looks like a bug, the number of iterations didn't change but sum did?
This value is coming in from the entry block and represents the initial value.  I changed it to 1 from 0 so that the identity operation would not apply and this test would not otherwise change.  I did this to the tests here because these tests are for testing other behaviours, rather than testing the peeling of an identity operator. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142255



More information about the llvm-commits mailing list