[PATCH] D93096: [llvm-readelf] - Improve ELF type field dumping.

[George Rimar via Phabricator via llvm-commits]

grimar added inline comments.


================
Comment at: llvm/tools/llvm-readobj/ELFDumper.cpp:3532
+  if (e.e_type >= ET_LOPROC) {
+    Str = "Processor Specific: (" + Str + ")";
+  } else if (e.e_type >= ET_LOOS) {
----------------
MaskRay wrote:
> I think the value is the main body and `Processor Specific` is a comment. If something needs to be placed in `()`, it should not be the value.
I am printing exactly what GNU readelf prints here.


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

https://reviews.llvm.org/D93096