[PATCH] D139097: [ARM] Add option --print-raw-value to llvm-nm to dump raw symbol values in case of ARM

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 31 09:57:22 PST 2023


MaskRay added a comment.

In D139097#4092831 <https://reviews.llvm.org/D139097#4092831>, @jhenderson wrote:

> In D139097#4073914 <https://reviews.llvm.org/D139097#4073914>, @peter.smith wrote:
>
>> So I think this still leaves us with the choice of whether llvm-nm should differ from arm-none-eabi-nm. I personally would err on the side of caution and consistency between GNU and LLVM and keep the default behaviour the same. However I don't think it will make a huge difference.
>
> I'm inclined to agree with @peter.smith re. compatibility. I don't think there's a strong enough motivation to diverge here, and I've seen llvm-nm used in scripts before to gather symbol data. This means we can't safely change the behaviour like we can with some other tools that are likely only for direct human consumption.
>
> I think this means the option has a place here. @MaskRay, any further thoughts?

No further thought. Adding an option seems the way to go.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139097/new/

https://reviews.llvm.org/D139097



More information about the llvm-commits mailing list