From: Michael Beck <mm.beck@gmx.net>
Date: Wed, 13 Apr 2011 13:37:03 +0000 (+0200)
Subject: BugFix: Fixed wrong todo fix.
X-Git-Url: http://nsz.repo.hu/git/?a=commitdiff_plain;h=42deb58bf4970b4f91e23e9b24309b6f3446b42a;p=libfirm

BugFix: Fixed wrong todo fix.
---

diff --git a/ir/ir/irop.c b/ir/ir/irop.c
index 82d2f144d..d4318f0fd 100644
--- a/ir/ir/irop.c
+++ b/ir/ir/irop.c
@@ -86,10 +86,11 @@ static void block_copy_attr(ir_graph *irg, const ir_node *old_node,
 	new_node->attr.block.block_visited = 0;
 	memset(&new_node->attr.block.dom, 0, sizeof(new_node->attr.block.dom));
 	memset(&new_node->attr.block.pdom, 0, sizeof(new_node->attr.block.pdom));
-	/* TODO: we should probably create a new entity. But we somehow have to
-	 * patch the stuff at the same time */
-	new_node->attr.block.entity            = NULL;
-	new_node->attr.block.phis              = NULL;
+	/* It should be safe to copy the entity here, as it has no back-link to the old block.
+	 * It serves just as a label number, so copying a labeled block results in an exact copy.
+	 * This is at least what we need for DCE to work. */
+	new_node->attr.block.entity         = old_node->attr.block.entity;
+	new_node->attr.block.phis           = NULL;
 	INIT_LIST_HEAD(&new_node->attr.block.succ_head);
 }