[PATCH] D57146: [llvm-objdump] - Print LMAs when dumping section headers.

[James Henderson via Phabricator via llvm-commits]

jhenderson added a comment.

In D57146#1370988 <https://reviews.llvm.org/D57146#1370988>, @grimar wrote:

> In D57146#1370968 <https://reviews.llvm.org/D57146#1370968>, @jhenderson wrote:
>
> > One further suggestion for 1) might be to change the header to VMA/LMA for the address column, to make it clearer. It might be less confusing to those used to GNU output?
>
>
> I thought about that too. Decided to not suggest it because if we hide a LMA column completely then probably no need to make people who are unfamiliar with it even to think about it?
>  I.e. This patch prints "VMA ... LMA" now and "Address" in another case. I supposed that just "Address" would look simpler then.
>
> I am not sure what would be really better though honestly. Maybe mixing "VMA" and "Address" for different cases is also not a very good idea.


Okay. I'd just stick with VMA when printing only that.


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

https://reviews.llvm.org/D57146