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

Subham Kedia via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 6 12:05:41 PST 2022


quic-subhkedi added a comment.

In D139097#3975462 <https://reviews.llvm.org/D139097#3975462>, @shankare wrote:

> it looks like with the change to the API to just return st_value, few bots broke, Subham ?
>
> May be just printing the raw value is an option since callers to this API expect the value returned  as per ARM/Mips ABI ?

Yes Shankar, an early revision of this patch tried to simply just emit st_value and it broke a bunch of tests across the board including clang and lld tests, adding an option is the safer way in my opinion


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

https://reviews.llvm.org/D139097



More information about the llvm-commits mailing list