[PATCH] D65537: Print reasonable representations of type names in llvm-nm, readelf and readobj.

[George Rimar via Phabricator via llvm-commits]

grimar added a comment.

I tested the output with the change I suggested and it was:

  Name                  Value           Class        Type         Size
   Line  Section
  os_specific_10       |                |   U  |             IFUNC|     |     |*UND*
  os_specific_11       |                |   U  | <OS specific>(11)|     |     |*UND*
  os_specific_12       |                |   U  | <OS specific>(12)|     |     |*UND*
  proc_specific_13     |                |   U  |<processor specific>(13)|     |     |*UND*
  proc_specific_14     |                |   U  |<processor specific>(14)|     |     |*UND*
  proc_specific_15     |                |   U  |<processor specific>(15)|     |     |*UND*
  symbol_common        |                |   U  |            COMMON|     |     |*COM*
  symbol_file          |                |   U  |              FILE|     |     |*UND*
  symbol_func          |                |   U  |              FUNC|     |     |*UND*
  symbol_ifunc         |                |   U  |             IFUNC|     |     |*UND*
  symbol_notype        |                |   U  |            NOTYPE|     |     |*UND*
  symbol_obj           |                |   U  |            OBJECT|     |     |*UND*
  symbol_tls           |                |   U  |               TLS|     |     |*UND*
  unknown_7            |                |   U  |      <unknown>(7)|     |     |*UND*
  unknown_8            |                |   U  |      <unknown>(8)|     |     |*UND*
  unknown_9            |                |   U  |      <unknown>(9)|

Looks good to me, though I do not really mind to keep the original output probably.
Lets see what others think.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D65537