[lld] r234144 - [AArch64] Fix error in AARCH64_ADR_GOT_PAGE handler

Rui Ueyama ruiu at google.com
Mon Apr 6 10:32:37 PDT 2015


It'd be nice if we can test this?

On Mon, Apr 6, 2015 at 12:44 AM, Denis Protivensky <
dprotivensky at accesssoftek.com> wrote:

> Author: denis-protivensky
> Date: Mon Apr  6 02:44:59 2015
> New Revision: 234144
>
> URL: http://llvm.org/viewvc/llvm-project?rev=234144&view=rev
> Log:
> [AArch64] Fix error in AARCH64_ADR_GOT_PAGE handler
>
> The error was introduced during mechanical replacement
> of raw memory reads/writes to use readxxle/writexxle functions
> in r230725.
>
> Noted and fixed by Suprateeka R Hegde <hegdesmailbox at gmail.com>
>
> Modified:
>     lld/trunk/lib/ReaderWriter/ELF/AArch64/AArch64RelocationHandler.cpp
>
> Modified:
> lld/trunk/lib/ReaderWriter/ELF/AArch64/AArch64RelocationHandler.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/ELF/AArch64/AArch64RelocationHandler.cpp?rev=234144&r1=234143&r2=234144&view=diff
>
> ==============================================================================
> --- lld/trunk/lib/ReaderWriter/ELF/AArch64/AArch64RelocationHandler.cpp
> (original)
> +++ lld/trunk/lib/ReaderWriter/ELF/AArch64/AArch64RelocationHandler.cpp
> Mon Apr  6 02:44:59 2015
> @@ -223,7 +223,7 @@ static void relocR_AARCH64_ADR_GOT_PAGE(
>          llvm::dbgs() << " immhi: " << Twine::utohexstr(immhi);
>          llvm::dbgs() << " immlo: " << Twine::utohexstr(immlo);
>          llvm::dbgs() << " result: " << Twine::utohexstr(result) << "\n");
> -  write32le(location, result | read32le(location));
> +  write32le(location, immlo | immhi | read32le(location));
>  }
>
>  // R_AARCH64_LD64_GOT_LO12_NC
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150406/2b0127d1/attachment.html>


More information about the llvm-commits mailing list