[llvm] r174717 - The patch to fix some issues in r174543 fixed the lines failing the test, but missed a couple

David Tweed david.tweed at arm.com
Fri Feb 8 08:35:11 PST 2013


Author: davidtweed
Date: Fri Feb  8 10:35:10 2013
New Revision: 174717

URL: http://llvm.org/viewvc/llvm-project?rev=174717&view=rev
Log:
The patch to fix some issues in r174543 fixed the lines failing the test, but missed a couple
of lines which weren't being explicitly looked at and were printing incorrect results. These
values clearly must lie within 32 bits, so the casts are definitely safe.

Modified:
    llvm/trunk/lib/DebugInfo/DWARFDebugFrame.cpp

Modified: llvm/trunk/lib/DebugInfo/DWARFDebugFrame.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/DebugInfo/DWARFDebugFrame.cpp?rev=174717&r1=174716&r2=174717&view=diff
==============================================================================
--- llvm/trunk/lib/DebugInfo/DWARFDebugFrame.cpp (original)
+++ llvm/trunk/lib/DebugInfo/DWARFDebugFrame.cpp Fri Feb  8 10:35:10 2013
@@ -69,9 +69,9 @@ public:
        << "\n";
     OS << format("  Version:               %d\n", Version);
     OS << "  Augmentation:          \"" << Augmentation << "\"\n";
-    OS << format("  Code alignment factor: %u\n", CodeAlignmentFactor);
-    OS << format("  Data alignment factor: %d\n", DataAlignmentFactor);
-    OS << format("  Return address column: %d\n", ReturnAddressRegister);
+    OS << format("  Code alignment factor: %u\n", (uint32_t)CodeAlignmentFactor);
+    OS << format("  Data alignment factor: %d\n", (int32_t)DataAlignmentFactor);
+    OS << format("  Return address column: %d\n", (int32_t)ReturnAddressRegister);
     OS << "\n";
   }
 





More information about the llvm-commits mailing list