[llvm-commits] [llvm] r104705 - /llvm/trunk/lib/CodeGen/AsmPrinter/DwarfDebug.cpp

[Devang Patel]

Author: dpatel
Date: Wed May 26 12:29:32 2010
New Revision: 104705

URL: http://llvm.org/viewvc/llvm-project?rev=104705&view=rev
Log:
Do not construct location list backword!

Modified:
    llvm/trunk/lib/CodeGen/AsmPrinter/DwarfDebug.cpp

Modified: llvm/trunk/lib/CodeGen/AsmPrinter/DwarfDebug.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/AsmPrinter/DwarfDebug.cpp?rev=104705&r1=104704&r2=104705&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/AsmPrinter/DwarfDebug.cpp (original)
+++ llvm/trunk/lib/CodeGen/AsmPrinter/DwarfDebug.cpp Wed May 26 12:29:32 2010
@@ -2215,22 +2215,26 @@
     RegVar->setDotDebugLocOffset(DotDebugLocEntries.size());
     if (DotDebugLocEntries.empty())
       DotDebugLocEntries.push_back(DotDebugLocEntry());
-    const MachineInstr *Current = MultipleValues.back(); 
-    MultipleValues.pop_back();
-    while (!MultipleValues.empty()) {
-      const MachineInstr *Next = MultipleValues.back();
-      MultipleValues.pop_back();
-      DbgValueStartMap[Next] = RegVar;
+    const MachineInstr *Begin = NULL;
+    const MachineInstr *End = NULL;
+    for (SmallVector<const MachineInstr *, 4>::iterator 
+           MVI = MultipleValues.begin(), MVE = MultipleValues.end(); MVI != MVE; ++MVI) {
+      if (!Begin) {
+        Begin = *MVI;
+        continue;
+      } 
+      End = *MVI;
+      DbgValueStartMap[End] = RegVar;
       MachineLocation MLoc;
-      MLoc.set(Current->getOperand(0).getReg(), 0);
-      const MCSymbol *FLabel = getLabelBeforeInsn(Next);
-      const MCSymbol *SLabel = getLabelBeforeInsn(Current);
+      MLoc.set(Begin->getOperand(0).getReg(), 0);
+      const MCSymbol *FLabel = getLabelBeforeInsn(Begin);
+      const MCSymbol *SLabel = getLabelBeforeInsn(End);
       DotDebugLocEntries.push_back(DotDebugLocEntry(FLabel, SLabel, MLoc));
-      Current = Next;
-      if (MultipleValues.empty()) {
-        // If Next is the last instruction then its value is valid
+      Begin = End;
+      if (MVI + 1 == MVE) {
+        // If End is the last instruction then its value is valid
         // until the end of the funtion.
-        MLoc.set(Next->getOperand(0).getReg(), 0);
+        MLoc.set(End->getOperand(0).getReg(), 0);
         DotDebugLocEntries.
           push_back(DotDebugLocEntry(SLabel, FunctionEndSym, MLoc));
       }