[llvm-commits] [LLVM, llvm-nm, arm] Fix for nm-trivial-object.test and for nm-archive.test

[Stepan Dyatkovskiy]

Hi,
Yes, you right. Sorry, I actually mean clang-native-arm-cortex-a9:
http://lab.llvm.org:8011/builders/clang-native-arm-cortex-a9/builds/155

Regards,
Stepan.

Michael Spencer wrote:
> On Fri, Oct 21, 2011 at 11:50 AM, Stepan Dyatkovskiy<stpworld at narod.ru>  wrote:
>> Hi all,
>> Please find the patch for review that fixes nm-trivial-object.test and
>> nm-archive.test for arm architecture.
>>
>> Regards,
>> Stepan
>
> These tests are currently passing on the llvm-arm-linux buildbot.
> Where are you encountering the failure?
>
> This does, however, show that the code is currently wrong. It's trying
> to print a uint64_t as an unsigned. The code needs to use the address
> size from the object file to print either 8 byte or 4 byte addresses
> and sizes using the correct format string.
>
> Thanks for bringing this up!
>
> - Michael Spencer