[PATCH] D28984: [LLD] Do not allocate space for common symbols with -r

[Mark Kettenis via Phabricator via llvm-commits]

kettenis added a comment.

h>>! In https://reviews.llvm.org/D28984#652498, @ruiu wrote:

> Can you add a test?

Sure.

The alignment is returned as the VA because that's what's being stored in the st_value field of SHN_COMMON symbols.
Alternatively I could return 0 here, and fix up st_value in SymbolTableSection<ELFT>::writeGlobalSymbols(), but this way
seems a little bit cleaner.


https://reviews.llvm.org/D28984