[PATCH] D57105: [ELF] Return the section name when calling getSymbolName on a section symbol.

Matt Davis via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 23 16:13:15 PST 2019


mattd planned changes to this revision.
mattd added a comment.

I have a few changes to this patch.  Namely, I want to avoid breaking the llvm-objdump/AArch64/elf-aarch64-mapping-symbols.test.  I didn't realize I had broken that test earlier, but I caught it in my second round of testing. It turns out that llvm-objdump presents a data mapping symbol ($x) when the test works.  However, with the patch above, that mapping symbol is gone and the section name is displayed instead.  I'm working on fixing my patch to avoid that situation.


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

https://reviews.llvm.org/D57105





More information about the llvm-commits mailing list