[llvm-commits] [llvm] r143120 - /llvm/trunk/tools/llvm-objdump/llvm-objdump.cpp

[Stepan Dyatkovskiy]

Hi, Takumi.

I attached fixes you ask. If all is OK, I can commit it again.

-Stepan.

NAKAMURA Takumi wrote:
> Stepan, thanks for working!
> 
> 2011年10月28日 3:40:45 UTC+9 Stepan Dyatkovskiy<stpworld at narod.ru>:
>> Author: dyatkovskiy
>> Date: Thu Oct 27 13:40:45 2011
>> New Revision: 143120
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=143120&view=rev
>> Log:
>> Fixed llvm-objdump uint64_t formatted output.
>>
>> Modified:
>>     llvm/trunk/tools/llvm-objdump/llvm-objdump.cpp
>>
>> Modified: llvm/trunk/tools/llvm-objdump/llvm-objdump.cpp
>> URL:
>> http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-objdump/llvm-objdump.cpp?rev=143120&r1=143119&r2=143120&view=diff
> 
>> -          outs()<<  format("%8x:\t", SectionAddr + Index);
>> +          outs()<<  format("%8llx:\t", SectionAddr + Index);
> (ditto)
> 
> Unfortunately, msvcrt does not support %ll.
> It works, though %llx is treated as %x, and msvc is on little-endian host.
> (Yeah, it had been failing on big-endian ppc-linux)
> 
> Could you tweak again with PRI_* macros? (You can see in DataTypes.h.cmake)
> 
> ...Takumi


-------------- next part --------------
A non-text attachment was scrubbed...
Name: uint64_output.patch
Type: text/x-patch
Size: 2788 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20111028/c28b07e4/attachment.bin>