[PATCH] D36835: [llvm-dwarfdump] Hide .debug_str and die reference offsets in brief mode

Jonas Devlieghere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 11 12:38:17 PDT 2017


JDevlieghere added a comment.

Hi Dave,

The positive tests have been committed as part of https://reviews.llvm.org/D37334 because I had to change the test anyway. Let me know if you think something is still missing!

With regards to dropping the prefixes, I talked to Adrian and he makes a pretty good argument in favor of keeping them. Maybe this is something to discuss on the mailing list as an NFC? I've had a look and implementing this should be relatively straightforward so it really boils down to whether we want to do this or not.


Repository:
  rL LLVM

https://reviews.llvm.org/D36835





More information about the llvm-commits mailing list