[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