[all-commits] [llvm/llvm-project] 018a48: [llvm-objdump] Map STT_TLS to ST_Other (previously...

[Fangrui Song via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 018a484cd26d72fb4c9e7fd75e5f5bc7838dfc73
      https://github.com/llvm/llvm-project/commit/018a484cd26d72fb4c9e7fd75e5f5bc7838dfc73
  Author: Fangrui Song <i at maskray.me>
  Date:   2021-02-17 (Wed, 17 Feb 2021)

  Changed paths:
    M lld/test/ELF/mips-mgot.s
    M lld/test/ELF/mips-tls-64.s
    M lld/test/ELF/mips-tls-hilo.s
    M lld/test/ELF/mips-tls-static-64.s
    M lld/test/ELF/mips-tls-static.s
    M lld/test/ELF/mips-tls.s
    M llvm/include/llvm/Object/ELFObjectFile.h
    M llvm/test/tools/llvm-objdump/ELF/symbol-table.test

  Log Message:
  -----------
  [llvm-objdump] Map STT_TLS to ST_Other (previously ST_Data)

ST_Data is used to model BFD `BFD_OBJECT`.
A STT_TLS symbol does not have the `BFD_OBJECT` flag in BFD.
This makes sense because a STT_TLS symbol is like in a different address space,
normal data/object properties do not apply on them.

With this change, a STT_TLS symbol will not be displayed as 'O'.
This new behavior matches objdump.

Differential Revision: https://reviews.llvm.org/D96735