[PATCH] D48472: [ELF] Change isSectionData to exclude SHF_EXECINSTR

[Fangrui Song via Phabricator via llvm-commits]

MaskRay added a comment.

I agree with jyknight on this.. "TEXT" "DATA" "BSS" add an unnecessary abstraction layer on top of SHT_PROGBITS SHF_EXECINSTR ... If we could start over, we should remove the abstraction and change the display.

Thankfully most of this usage is superseded by llvm-readelf . Chime in freebsd hackers as there is recently discussion on moving away from GNU binutils https://lists.freebsd.org/pipermail/freebsd-toolchain/2018-June/003809.html


Repository:
  rL LLVM

https://reviews.llvm.org/D48472