From: Benjamin Kramer <benny.kra@googlemail.com>
Date: Fri, 16 Mar 2012 17:38:19 +0000 (+0000)
Subject: ScheduleDAGInstrs: When adding uses we add them into a set that's empty at the beginn... 
X-Git-Url: http://plrg.eecs.uci.edu/git/?a=commitdiff_plain;h=a82d526b31cd9825684fdee5c3372bf55dad4f57;p=oota-llvm.git

ScheduleDAGInstrs: When adding uses we add them into a set that's empty at the beginning, no need to maintain another set for the added regs.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@152934 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/CodeGen/ScheduleDAGInstrs.cpp b/lib/CodeGen/ScheduleDAGInstrs.cpp
index 1c455b95ab0..6be1ab7f5b0 100644
--- a/lib/CodeGen/ScheduleDAGInstrs.cpp
+++ b/lib/CodeGen/ScheduleDAGInstrs.cpp
@@ -209,13 +209,13 @@ void ScheduleDAGInstrs::addSchedBarrierDeps() {
   } else {
     // For others, e.g. fallthrough, conditional branch, assume the exit
     // uses all the registers that are livein to the successor blocks.
-    SmallSet<unsigned, 8> Seen;
+    assert(Uses.empty() && "Uses in set before adding deps?");
     for (MachineBasicBlock::succ_iterator SI = BB->succ_begin(),
            SE = BB->succ_end(); SI != SE; ++SI)
       for (MachineBasicBlock::livein_iterator I = (*SI)->livein_begin(),
              E = (*SI)->livein_end(); I != E; ++I) {
         unsigned Reg = *I;
-        if (Seen.insert(Reg))
+        if (!Uses.contains(Reg))
           Uses[Reg].push_back(&ExitSU);
       }
   }