[PATCH] D24910: [ELF] - Linkerscript: implemented LOADADDR()

[Eugene Leviant via llvm-commits]

evgeny777 added inline comments.

================
Comment at: ELF/LinkerScript.cpp:732
@@ +731,3 @@
+uint64_t LinkerScript<ELFT>::getOutputSectionLoadAddress(StringRef Name) {
+  uint64_t VA = getOutputSectionAddress(Name);
+  if (Expr LmaExpr = getLma(Name))
----------------
Looks like this is not correct.
>From https://sourceware.org/binutils/docs-2.20/ld/Output-Section-LMA.html

> If neither AT nor AT> is specified for an allocatable section, the linker will set the LMA such that the difference between VMA and LMA for the section is the same as the preceding output section in the same region




https://reviews.llvm.org/D24910