[PATCH] D61117: Fix Bug 41353 - unique symbols printed as D instead of u

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 6 04:37:51 PDT 2019


grimar added a comment.

In D61117#1488853 <https://reviews.llvm.org/D61117#1488853>, @rupprecht wrote:

> Looks like support was recently added in rL359380 <https://reviews.llvm.org/rL359380>, though without any tests, and was accidentally reverted.


I think it just had a bug. Binding is `st_info >> 4`, not `st_other >> 4`:
https://github.com/llvm-mirror/llvm/blob/master/include/llvm/BinaryFormat/ELF.h#L995


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61117





More information about the llvm-commits mailing list