<p dir="ltr"><br>
On Mar 10, 2016 6:30 PM, "Rui Ueyama" <<a href="mailto:ruiu@google.com">ruiu@google.com</a>> wrote:<br>
><br>
> ruiu added inline comments.<br>
><br>
> ================<br>
> Comment at: ELF/OutputSections.cpp:272-277<br>
> @@ +271,8 @@<br>
> +        if (auto *L = dyn_cast<LocalSymbol<ELFT>>(Sym)) {<br>
> +          uintX_t Pos = L->Sym.st_value;<br>
> +          if (L->Sym.getType() == STT_SECTION) {<br>
> +            Pos += Addend;<br>
> +            Addend = 0;<br>
> +          }<br>
> +          VA = L->Section->OutSec->getVA() + L->Section->getOffset(Pos);<br>
> +        } else {<br>
> ----------------<br>
> Can't you move this to getVA for LocalSymbol?<br>
><br>
></p>
<p dir="ltr">That is what the next patch does:<br>
<a href="http://reviews.llvm.org/D18056">http://reviews.llvm.org/D18056</a><br>
</p>