[PATCH] D44899: [ELF] - Print LMA in a -Map file.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 28 02:24:04 PDT 2018


grimar added a comment.

In https://reviews.llvm.org/D44899#1050199, @jhenderson wrote:

> Do we always want to show LMAs and VMAs? In some (possibly many) systems, the LMA will always match the VMA, so it seems like a lot of noise that doesn't benefit many consumers. Perhaps this could be put under an extra switch? Something like "--show-lma-in-map" (happy with a better name)?


I suggested a bit different thing in the description that might help to solve this:
"to hide LMA column if it is known there is no difference with VA column. Then VA could be renamed to VMA/LMA"

That might not be convenient for parsing sometimes probably since we would have a different amount of columns depending on inputs.
Not sure how much it is a problem though.

FWIW, Rui's suggestion to drop leading zeroes also looks fine for me, it should make map file more readable with both VMA/LMA I think.


https://reviews.llvm.org/D44899





More information about the llvm-commits mailing list