[PATCH] D17523: llvm-readobj: enable GNU output style sections and relocations printing for ELF files

khemant@codeaurora.org via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 23 15:49:09 PST 2016


khemant added inline comments.

================
Comment at: tools/llvm-readobj/ELFDumper.cpp:523
@@ -459,3 +522,3 @@
                                             const Elf_Sym *symb,
-                                            bool &IsDefault) {
+                                            bool &IsDefault) const {
   // This is a dynamic symbol. Look in the GNU symbol version table.
----------------
This is the patch that introduced the pointer. I would like to have it const from beginning instead of retrofitting in future as it will cause larger unneeded churn.


Repository:
  rL LLVM

http://reviews.llvm.org/D17523





More information about the llvm-commits mailing list