[PATCH] D72234: [llvm-readelf] Print EI_ABIVERSION as decimal instead of hexadecimal
    James Henderson via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Jan  6 01:49:51 PST 2020
    
    
  
jhenderson accepted this revision.
jhenderson added a comment.
This revision is now accepted and ready to land.
LGTM. I have no idea why this was hex in the first place.
================
Comment at: llvm/tools/llvm-readobj/ELFDumper.cpp:3065
+  printFields(OS,
+              "ABI Version:", std::to_string(e->e_ident[ELF::EI_ABIVERSION]));
   Str = printEnum(e->e_type, makeArrayRef(ElfObjectFileType));
----------------
BTW, do you know the rationale for using llvm::to_string versus std::to_string? Both are widely used in the code base...
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D72234/new/
https://reviews.llvm.org/D72234
    
    
More information about the llvm-commits
mailing list