[llvm] r185361 - Don't form PPC CTR loops for over-sized exit counts

Hal Finkel hfinkel at anl.gov
Mon Jul 1 12:34:59 PDT 2013


Author: hfinkel
Date: Mon Jul  1 14:34:59 2013
New Revision: 185361

URL: http://llvm.org/viewvc/llvm-project?rev=185361&view=rev
Log:
Don't form PPC CTR loops for over-sized exit counts

Although you can't generate this from C on PPC64, if you have a loop using a
64-bit counter on PPC32 then you can't form a CTR-based loop for it. This had
been cauing the PPCCTRLoops pass to assert.

Thanks to Joerg Sonnenberger for providing a test case!

Added:
    llvm/trunk/test/CodeGen/PowerPC/ctrloop-large-ec.ll
Modified:
    llvm/trunk/lib/Target/PowerPC/PPCCTRLoops.cpp

Modified: llvm/trunk/lib/Target/PowerPC/PPCCTRLoops.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/PowerPC/PPCCTRLoops.cpp?rev=185361&r1=185360&r2=185361&view=diff
==============================================================================
--- llvm/trunk/lib/Target/PowerPC/PPCCTRLoops.cpp (original)
+++ llvm/trunk/lib/Target/PowerPC/PPCCTRLoops.cpp Mon Jul  1 14:34:59 2013
@@ -415,6 +415,9 @@ bool PPCCTRLoops::convertToCTRLoop(Loop
     } else if (!SE->isLoopInvariant(EC, L))
       continue;
 
+    if (SE->getTypeSizeInBits(EC->getType()) > (TT.isArch64Bit() ? 64 : 32))
+      continue;
+
     // We now have a loop-invariant count of loop iterations (which is not the
     // constant zero) for which we know that this loop will not exit via this
     // exisiting block.

Added: llvm/trunk/test/CodeGen/PowerPC/ctrloop-large-ec.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/PowerPC/ctrloop-large-ec.ll?rev=185361&view=auto
==============================================================================
--- llvm/trunk/test/CodeGen/PowerPC/ctrloop-large-ec.ll (added)
+++ llvm/trunk/test/CodeGen/PowerPC/ctrloop-large-ec.ll Mon Jul  1 14:34:59 2013
@@ -0,0 +1,23 @@
+; RUN: llc -mcpu=ppc32 < %s
+target datalayout = "E-p:32:32:32-i1:8:8-i8:8:8-i16:16:16-i32:32:32-i64:64:64-f32:32:32-f64:64:64-v128:128:128-n32"
+target triple = "powerpc-unknown-linux-gnu"
+
+define void @fn1() {
+entry:
+  br i1 undef, label %for.end, label %for.body
+
+for.body:                                         ; preds = %for.body, %entry
+  %inc3 = phi i64 [ %inc, %for.body ], [ undef, %entry ]
+  %inc = add nsw i64 %inc3, 1
+  %tobool = icmp eq i64 %inc, 0
+  br i1 %tobool, label %for.end, label %for.body
+
+for.end:                                          ; preds = %for.body, %entry
+  ret void
+}
+
+; On PPC32, CTR is also 32 bits, and so cannot hold a 64-bit count.
+; CHECK: @fn1
+; CHECK-NOT: mtctr
+; CHECK: blr
+





More information about the llvm-commits mailing list