[PATCH] D59095: [llvm-readelf]Don't lose negative-ness of negative addends for no symbol relocations

Xing via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 7 16:57:45 PST 2019


Higuoxing added a comment.

LGTM.



================
Comment at: tools/llvm-readobj/ELFDumper.cpp:3401
+    }
+    Addend += to_string(format_hex_no_prefix(RelAddend, 1));
   }
----------------
Can we use `to_hexString(RelAddend, false)` here? I think it's better to have consistent code style.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59095





More information about the llvm-commits mailing list