[PATCH] D33226: [NewGVN] Replace predicate info leftovers

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 15 22:36:53 PDT 2017


This revision was automatically updated to reflect the committed changes.
Closed by commit rL303141: [NewGVN] Replace predicate info leftovers. (authored by davide).

Changed prior to commit:
  https://reviews.llvm.org/D33226?vs=99098&id=99102#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D33226

Files:
  llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp
  llvm/trunk/test/Transforms/NewGVN/pr32945.ll


Index: llvm/trunk/test/Transforms/NewGVN/pr32945.ll
===================================================================
--- llvm/trunk/test/Transforms/NewGVN/pr32945.ll
+++ llvm/trunk/test/Transforms/NewGVN/pr32945.ll
@@ -0,0 +1,24 @@
+; RUN: opt -S -newgvn %s | FileCheck %s
+; CHECK-NOT: call i32 @llvm.ssa.copy
+
+ at d = external global i32
+ at e = external global i32
+define void @tinkywinky() {
+  br i1 true, label %lor.lhs.false, label %cond.true
+lor.lhs.false:
+  %tmp = load i32, i32* @d, align 4
+  %patatino = load i32, i32* null, align 4
+  %or = or i32 %tmp, %patatino
+  store i32 %or, i32* @d, align 4
+  br label %cond.true
+cond.true:
+  %tmp1 = load i32, i32* @e, align 4
+  %tmp2 = load i32, i32* @d, align 4
+  %cmp = icmp eq i32 %tmp1, %tmp2
+  br i1 %cmp, label %cond.true6, label %cond.false
+cond.true6:
+  %cmp7 = icmp slt i32 %tmp1, 0
+  br i1 %cmp7, label %cond.false, label %cond.false
+cond.false:
+  ret void
+}
Index: llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp
===================================================================
--- llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp
+++ llvm/trunk/lib/Transforms/Scalar/NewGVN.cpp
@@ -3291,6 +3291,10 @@
 
           Value *DominatingLeader = EliminationStack.back();
 
+          auto *II = dyn_cast<IntrinsicInst>(DominatingLeader);
+          if (II && II->getIntrinsicID() == Intrinsic::ssa_copy)
+            DominatingLeader = II->getOperand(0);
+
           // Don't replace our existing users with ourselves.
           if (U->get() == DominatingLeader)
             continue;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D33226.99102.patch
Type: text/x-patch
Size: 1559 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170516/669a7f2f/attachment.bin>


More information about the llvm-commits mailing list