[PATCH] D41403: [ELF] Fix incorrect physical address on self-referencing AT command

Erick Reyes via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 9 18:04:28 PST 2018


Just reviewed Rafael's patch and confirmed it also fixes the issue and
passes all tests on r322147.

However, I can't tell if the change in switchTo() can have side effects
since this function is called in LinkerScript::assignAddresses().



On Tue, Jan 9, 2018 at 5:34 PM, Rui Ueyama via Phabricator <
reviews at reviews.llvm.org> wrote:

> ruiu accepted this revision.
> ruiu added a comment.
> This revision is now accepted and ready to land.
>
> LGTM
>
>
> Repository:
>   rLLD LLVM Linker
>
> https://reviews.llvm.org/D41403
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180109/af2394ee/attachment.html>


More information about the llvm-commits mailing list