<div dir="ltr">Yes, I am writing test cases for this. The test case needs to be put under <span style="font-size:13.1999998092651px;line-height:19.7999992370605px">test/elf/AArch64 in lld sources. I will post the test cases soon here with all the details.</span><br><br>Thanks a lot<div>--</div><div>Supra<br><br><div class="gmail_quote">On Tue, 7 Apr 2015 at 11:59 Denis Protivensky <<a href="mailto:dprotivensky@accesssoftek.com" target="_blank">dprotivensky@accesssoftek.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000">
Yep, the original author of the fix claimed he'd write the test case
soon. I obviously don't know how to properly test this, and I
applied the fix just because I was able to diff with previous
changes and find that there was an error. It's still better than
nothing.</div><div bgcolor="#FFFFFF" text="#000000"><br>
<br>
<div>On 04/06/2015 08:32 PM, Rui Ueyama
wrote:<br>
</div>
<blockquote type="cite">
<div dir="ltr">It'd be nice if we can test this?</div>
<div class="gmail_extra"><br>
<div class="gmail_quote">On Mon, Apr 6, 2015 at 12:44 AM, Denis
Protivensky <span dir="ltr"><<a href="mailto:dprotivensky@accesssoftek.com" target="_blank">dprotivensky@accesssoftek.com</a><u></u>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author:
denis-protivensky<br>
Date: Mon Apr 6 02:44:59 2015<br>
New Revision: 234144<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=234144&view=rev" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project?rev=234144&view=rev</a><br>
Log:<br>
[AArch64] Fix error in AARCH64_ADR_GOT_PAGE handler<br>
<br>
The error was introduced during mechanical replacement<br>
of raw memory reads/writes to use readxxle/writexxle
functions<br>
in r230725.<br>
<br>
Noted and fixed by Suprateeka R Hegde <<a href="mailto:hegdesmailbox@gmail.com" target="_blank">hegdesmailbox@gmail.com</a>><br>
<br>
Modified:<br>
lld/trunk/lib/ReaderWriter/<u></u>ELF/AArch64/<u></u>AArch64RelocationHandler.cpp<br>
<br>
Modified:
lld/trunk/lib/ReaderWriter/<u></u>ELF/AArch64/<u></u>AArch64RelocationHandler.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/ELF/AArch64/AArch64RelocationHandler.cpp?rev=234144&r1=234143&r2=234144&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/lld/trunk/lib/<u></u>ReaderWriter/ELF/AArch64/<u></u>AArch64RelocationHandler.cpp?<u></u>rev=234144&r1=234143&r2=<u></u>234144&view=diff</a><br>
==============================<u></u>==============================<u></u>==================<br>
---
lld/trunk/lib/ReaderWriter/<u></u>ELF/AArch64/<u></u>AArch64RelocationHandler.cpp
(original)<br>
+++
lld/trunk/lib/ReaderWriter/<u></u>ELF/AArch64/<u></u>AArch64RelocationHandler.cpp
Mon Apr 6 02:44:59 2015<br>
@@ -223,7 +223,7 @@ static void relocR_AARCH64_ADR_GOT_PAGE(<br>
llvm::dbgs() << " immhi: " <<
Twine::utohexstr(immhi);<br>
llvm::dbgs() << " immlo: " <<
Twine::utohexstr(immlo);<br>
llvm::dbgs() << " result: " <<
Twine::utohexstr(result) << "\n");<br>
- write32le(location, result | read32le(location));<br>
+ write32le(location, immlo | immhi | read32le(location));<br>
}<br>
<br>
// R_AARCH64_LD64_GOT_LO12_NC<br>
<br>
<br>
______________________________<u></u>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
</blockquote>
</div>
<br>
</div>
</blockquote>
<br>
</div>
______________________________<u></u><u></u>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailm<u></u>an/listinfo/llvm-commits</a><br>
</blockquote></div></div></div>