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

[Stepan Dyatkovskiy]

Hi all. Please find the reworked patch in attachment.

Regards,
Stepan.

Stepan Dyatkovskiy wrote:
> 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
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

-------------- next part --------------
A non-text attachment was scrubbed...
Name: trivial-object.patch
Type: text/x-patch
Size: 765 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20111024/2d134954/attachment.bin>