No subject

matze at braunis.de matze at braunis.de
Thu Aug 29 15:48:10 PDT 2013


>From 18b422ccf205d45406a9f3d4dffaa8868de6e50b Mon Sep 17 00:00:00 2001
Message-Id: <18b422ccf205d45406a9f3d4dffaa8868de6e50b.1377816463.git.matze at braunis.de>
In-Reply-To: <02d89dcc988d30d6b07e999ebfde0a81592c0dcb.1377816463.git.matze at braunis.de>
References: <02d89dcc988d30d6b07e999ebfde0a81592c0dcb.1377816463.git.matze at braunis.de>
From: Matthias Braun <matze at braunis.de>
Date: Mon, 26 Aug 2013 15:15:27 -0700
Subject: [PATCH 2/5] remove pointless assert
To: llvm-commits at cs.uiuc.edu

The if above it ensures the property anyway.
---
 lib/CodeGen/RegisterCoalescer.cpp | 2 --
 1 file changed, 2 deletions(-)

diff --git a/lib/CodeGen/RegisterCoalescer.cpp b/lib/CodeGen/RegisterCoalescer.cpp
index 76a4630..7bfb54d 100644
--- a/lib/CodeGen/RegisterCoalescer.cpp
+++ b/lib/CodeGen/RegisterCoalescer.cpp
@@ -583,8 +583,6 @@ bool RegisterCoalescer::removeCopyByCommutingDef(const CoalescerPair &CP,
   if (!BValNo || BValNo->def != CopyIdx)
     return false;
 
-  assert(BValNo->def == CopyIdx && "Copy doesn't define the value?");
-
   // AValNo is the value number in A that defines the copy, A3 in the example.
   VNInfo *AValNo = IntA.getVNInfoAt(CopyIdx.getRegSlot(true));
   assert(AValNo && "COPY source not live");
-- 
1.8.3.4




More information about the llvm-commits mailing list