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

[Stepan Dyatkovskiy]

ping.

Regards,
Stepan.

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