[llvm] r273132 - [LoopIdiom] Don't remove dead operands manually
David Majnemer via llvm-commits
llvm-commits at lists.llvm.org
Sun Jun 19 19:33:29 PDT 2016
Author: majnemer
Date: Sun Jun 19 21:33:29 2016
New Revision: 273132
URL: http://llvm.org/viewvc/llvm-project?rev=273132&view=rev
Log:
[LoopIdiom] Don't remove dead operands manually
Removing dead instructions requires remembering which operands have
already been removed. RecursivelyDeleteTriviallyDeadInstructions has
this logic, don't partially reimplement it in LoopIdiomRecognize.
This fixes PR28196.
Added:
llvm/trunk/test/Transforms/LoopIdiom/pr28196.ll
Modified:
llvm/trunk/lib/Transforms/Scalar/LoopIdiomRecognize.cpp
Modified: llvm/trunk/lib/Transforms/Scalar/LoopIdiomRecognize.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Scalar/LoopIdiomRecognize.cpp?rev=273132&r1=273131&r2=273132&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/Scalar/LoopIdiomRecognize.cpp (original)
+++ llvm/trunk/lib/Transforms/Scalar/LoopIdiomRecognize.cpp Sun Jun 19 21:33:29 2016
@@ -159,8 +159,7 @@ static void deleteDeadInstruction(Instru
SmallVector<Value *, 16> Operands(I->value_op_begin(), I->value_op_end());
I->replaceAllUsesWith(UndefValue::get(I->getType()));
I->eraseFromParent();
- for (Value *Op : Operands)
- RecursivelyDeleteTriviallyDeadInstructions(Op, TLI);
+ RecursivelyDeleteTriviallyDeadInstructions(I, TLI);
}
//===----------------------------------------------------------------------===//
Added: llvm/trunk/test/Transforms/LoopIdiom/pr28196.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Transforms/LoopIdiom/pr28196.ll?rev=273132&view=auto
==============================================================================
--- llvm/trunk/test/Transforms/LoopIdiom/pr28196.ll (added)
+++ llvm/trunk/test/Transforms/LoopIdiom/pr28196.ll Sun Jun 19 21:33:29 2016
@@ -0,0 +1,26 @@
+; RUN: opt -loop-idiom -S < %s | FileCheck %s
+
+target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"
+target triple = "x86_64-unknown-linux-gnu"
+
+define void @test1() {
+entry:
+ br label %for.body.preheader
+
+for.body.preheader: ; preds = %for.cond
+ br label %for.body
+
+for.body: ; preds = %for.body, %for.body.preheader
+ %indvars.iv = phi i32 [ 0, %for.body.preheader ], [ %indvars.iv.next, %for.body ]
+ %add.ptr3 = getelementptr inbounds i32, i32* null, i32 %indvars.iv
+ %add.ptr4 = getelementptr inbounds i32, i32* %add.ptr3, i32 1
+ %0 = load i32, i32* %add.ptr4, align 4
+ store i32 %0, i32* %add.ptr3, align 4
+ %indvars.iv.next = add nsw i32 %indvars.iv, 1
+ %exitcond = icmp ne i32 %indvars.iv.next, 6
+ br i1 %exitcond, label %for.body, label %for.body.preheader
+}
+
+; CHECK-LABEL: define void @test1(
+; CHECK: call void @llvm.memcpy.p0i8.p0i8.i64(i8* null, i8* inttoptr (i64 4 to i8*), i64 24, i32 4, i1 false)
+; CHECK-NOT: store
More information about the llvm-commits
mailing list