[llvm] r203031 - Remove special case in the DIEValue printing since it only existed

Eric Christopher echristo at gmail.com
Wed Mar 5 16:00:50 PST 2014


Author: echristo
Date: Wed Mar  5 18:00:49 2014
New Revision: 203031

URL: http://llvm.org/viewvc/llvm-project?rev=203031&view=rev
Log:
Remove special case in the DIEValue printing since it only existed
for verbose asm.

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=203031&r1=203030&r2=203031&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/AsmPrinter/DwarfDebug.cpp (original)
+++ llvm/trunk/lib/CodeGen/AsmPrinter/DwarfDebug.cpp Wed Mar  5 18:00:49 2014
@@ -1996,8 +1996,12 @@ void DwarfDebug::emitDIE(DIE *Die) {
     dwarf::Form Form = AbbrevData[i].getForm();
     assert(Form && "Too many attributes for DIE (check abbreviation)");
 
-    if (Asm->isVerbose())
+    if (Asm->isVerbose()) {
       Asm->OutStreamer.AddComment(dwarf::AttributeString(Attr));
+      if (Attr == dwarf::DW_AT_accessibility)
+        Asm->OutStreamer.AddComment(dwarf::AccessibilityString(
+            cast<DIEInteger>(Values[i])->getValue()));
+    }
 
     switch (Attr) {
     case dwarf::DW_AT_abstract_origin:
@@ -2032,14 +2036,6 @@ void DwarfDebug::emitDIE(DIE *Die) {
       }
       break;
     }
-    case dwarf::DW_AT_accessibility: {
-      if (Asm->isVerbose()) {
-        DIEInteger *V = cast<DIEInteger>(Values[i]);
-        Asm->OutStreamer.AddComment(dwarf::AccessibilityString(V->getValue()));
-      }
-      Values[i]->EmitValue(Asm, Form);
-      break;
-    }
     default:
       // Emit an attribute using the defined form.
       Values[i]->EmitValue(Asm, Form);





More information about the llvm-commits mailing list