<div dir="ltr">Hi Geoff,<div><br></div><div>This commit caused a 10% regression in huffbench: <a href="http://llvm.org/perf/db_default/v4/nts/daily_report/2016/3/1?filter-machine-regex=aarch64%7Carm%7Cgreen">http://llvm.org/perf/db_default/v4/nts/daily_report/2016/3/1?filter-machine-regex=aarch64%7Carm%7Cgreen</a></div><div><br></div><div>There are also many other tests in that report that have regressed that I suspect to be the same revision. We've also seen quite large swings (not all negative!) on internal/third party tests.</div><div><br></div><div>Could you please take a look at what could be going on?</div><div><br></div><div>Cheers,</div><div><br></div><div>James</div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, 29 Feb 2016 at 19:57 Geoff Berry via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: gberry<br>
Date: Mon Feb 29 13:53:22 2016<br>
New Revision: 262248<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=262248&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=262248&view=rev</a><br>
Log:<br>
[AArch64] Fix isLegalAddImmediate() to return true for valid negative values.<br>
<br>
Reviewers: t.p.northover, jmolloy<br>
<br>
Subscribers: mcrosier, aemerson, llvm-commits, rengolin<br>
<br>
Differential Revision: <a href="http://reviews.llvm.org/D17463" rel="noreferrer" target="_blank">http://reviews.llvm.org/D17463</a><br>
<br>
Added:<br>
    llvm/trunk/test/CodeGen/AArch64/neg-imm.ll<br>
Modified:<br>
    llvm/trunk/lib/Target/AArch64/AArch64ISelLowering.cpp<br>
<br>
Modified: llvm/trunk/lib/Target/AArch64/AArch64ISelLowering.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AArch64/AArch64ISelLowering.cpp?rev=262248&r1=262247&r2=262248&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AArch64/AArch64ISelLowering.cpp?rev=262248&r1=262247&r2=262248&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Target/AArch64/AArch64ISelLowering.cpp (original)<br>
+++ llvm/trunk/lib/Target/AArch64/AArch64ISelLowering.cpp Mon Feb 29 13:53:22 2016<br>
@@ -7280,6 +7280,8 @@ EVT AArch64TargetLowering::getOptimalMem<br>
<br>
 // 12-bit optionally shifted immediates are legal for adds.<br>
 bool AArch64TargetLowering::isLegalAddImmediate(int64_t Immed) const {<br>
+  // Same encoding for add/sub, just flip the sign.<br>
+  Immed = std::abs(Immed);<br>
   if ((Immed >> 12) == 0 || ((Immed & 0xfff) == 0 && Immed >> 24 == 0))<br>
     return true;<br>
   return false;<br>
@@ -7288,8 +7290,6 @@ bool AArch64TargetLowering::isLegalAddIm<br>
 // Integer comparisons are implemented with ADDS/SUBS, so the range of valid<br>
 // immediates is the same as for an add or a sub.<br>
 bool AArch64TargetLowering::isLegalICmpImmediate(int64_t Immed) const {<br>
-  if (Immed < 0)<br>
-    Immed *= -1;<br>
   return isLegalAddImmediate(Immed);<br>
 }<br>
<br>
<br>
Added: llvm/trunk/test/CodeGen/AArch64/neg-imm.ll<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/AArch64/neg-imm.ll?rev=262248&view=auto" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/AArch64/neg-imm.ll?rev=262248&view=auto</a><br>
==============================================================================<br>
--- llvm/trunk/test/CodeGen/AArch64/neg-imm.ll (added)<br>
+++ llvm/trunk/test/CodeGen/AArch64/neg-imm.ll Mon Feb 29 13:53:22 2016<br>
@@ -0,0 +1,46 @@<br>
+; RUN: llc -mtriple=aarch64-linux-gnu -verify-machineinstrs -o - %s | FileCheck %s<br>
+; LSR used to pick a sub-optimal solution due to the target responding<br>
+; conservatively to isLegalAddImmediate for negative values.<br>
+<br>
+declare void @foo(i32)<br>
+<br>
+define void @test(i32 %px) {<br>
+; CHECK-LABEL: test:<br>
+; CHECK: // %entry<br>
+; CHECK: subs<br>
+; CHECK-NEXT: csel<br>
+entry:<br>
+  %sub = add nsw i32 %px, -1<br>
+  %cmp = icmp slt i32 %px, 1<br>
+  %.sub = select i1 %cmp, i32 0, i32 %sub<br>
+  br label %for.body<br>
+<br>
+for.body:<br>
+; CHECK: // %for.body<br>
+; CHECK:  cmp<br>
+; CHECK-NEXT:  b.eq<br>
+; CHECK: // %if.then3<br>
+  %x.015 = phi i32 [ %inc, %for.inc ], [ %.sub, %entry ]<br>
+  %cmp2 = icmp eq i32 %x.015, %px<br>
+  br i1 %cmp2, label %for.inc, label %if.then3<br>
+<br>
+if.then3:<br>
+  tail call void @foo(i32 %x.015)<br>
+  br label %for.inc<br>
+<br>
+for.inc:<br>
+; CHECK: // %for.inc<br>
+; CHECK:  add<br>
+; CHECK-NEXT:  cmp<br>
+; CHECK:  b.le<br>
+; CHECK: // %for.cond.cleanup<br>
+  %inc = add nsw i32 %x.015, 1<br>
+  %cmp1 = icmp sgt i32 %x.015, %px<br>
+  br i1 %cmp1, label %for.cond.cleanup.loopexit, label %for.body<br>
+<br>
+for.cond.cleanup.loopexit:<br>
+  br label %for.cond.cleanup<br>
+<br>
+for.cond.cleanup:<br>
+  ret void<br>
+}<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>