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

[NAKAMURA Takumi]

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