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

Michael Spencer bigcheesegs at gmail.com
Tue Oct 25 15:48:50 PDT 2011


On Tue, Oct 25, 2011 at 10:21 AM, Stepan Dyatkovskiy <stpworld at narod.ru> wrote:
> ping.
>
> Regards,
> Stepan.
>
> Stepan Dyatkovskiy wrote:
>> Hi all. Please find the reworked patch in attachment.
>>
>> Regards,
>> Stepan.

Committed as r142984. Thanks!

- Michael Spencer

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



More information about the llvm-commits mailing list