<div dir="ltr">So the problem here appears to be that this tsan test is adding a huge constant to the address of a global as part of testing whether it is in shadow memory. Once the constant gets folded into the global address it goes out of range of the PREL relocation and the linker is unable to relocate it.<div><br></div><div>Probably the right fix here is to prevent an offset from being folded into a global's address which would take it out of bounds of the memory used by the global. I'll start looking at that.</div><div><br></div><div>Peter</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 10, 2018 at 9:35 AM, Yvan Roux <span dir="ltr"><<a href="mailto:yvan.roux@linaro.org" target="_blank">yvan.roux@linaro.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ok, Thanks Peter.<br>
<div class="HOEnZb"><div class="h5"><br>
On 10 April 2018 at 18:22, Peter Collingbourne <<a href="mailto:peter@pcc.me.uk">peter@pcc.me.uk</a>> wrote:<br>
> Yes, I've reverted in r329718 and will investigate later.<br>
><br>
> Peter<br>
><br>
> On Tue, Apr 10, 2018 at 7:08 AM, Yvan Roux <<a href="mailto:yvan.roux@linaro.org">yvan.roux@linaro.org</a>> wrote:<br>
>><br>
>> Hi Peter,<br>
>><br>
>> On 9 April 2018 at 22:03, Peter Collingbourne via Phabricator via<br>
>> llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
>> > This revision was automatically updated to reflect the committed<br>
>> > changes.<br>
>> > Closed by commit rL329611: AArch64: Allow offsets to be folded into<br>
>> > addresses with ELF. (authored by pcc, committed by ).<br>
>> ><br>
>> > Changed prior to commit:<br>
>> >   <a href="https://reviews.llvm.org/D45199?vs=140738&id=141722#toc" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D45199?vs=140738&id=141722#toc</a><br>
>> ><br>
>> > Repository:<br>
>> >   rL LLVM<br>
>> ><br>
>> > <a href="https://reviews.llvm.org/D45199" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D45199</a><br>
>> ><br>
>> > Files:<br>
>> >   llvm/trunk/lib/Target/AArch64/<wbr>AArch64ISelDAGToDAG.cpp<br>
>> >   llvm/trunk/lib/Target/AArch64/<wbr>AArch64ISelLowering.cpp<br>
>> >   llvm/trunk/test/CodeGen/<wbr>AArch64/arm64-2012-05-07-<wbr>MemcpyAlignBug.ll<br>
>> >   llvm/trunk/test/CodeGen/<wbr>AArch64/arm64-addrmode.ll<br>
>> >   llvm/trunk/test/CodeGen/<wbr>AArch64/arm64-atomic-128.ll<br>
>> >   llvm/trunk/test/CodeGen/<wbr>AArch64/arm64-memcpy-inline.ll<br>
>> >   llvm/trunk/test/CodeGen/<wbr>AArch64/arm64-misched-<wbr>multimmo.ll<br>
>> >   llvm/trunk/test/CodeGen/<wbr>AArch64/arm64-variadic-aapcs.<wbr>ll<br>
>> >   llvm/trunk/test/CodeGen/<wbr>AArch64/arm64-vector-ldst.ll<br>
>> >   llvm/trunk/test/CodeGen/<wbr>AArch64/cmpxchg-O0.ll<br>
>> >   llvm/trunk/test/CodeGen/<wbr>AArch64/func-argpassing.ll<br>
>> >   llvm/trunk/test/CodeGen/<wbr>AArch64/func-calls.ll<br>
>> >   llvm/trunk/test/CodeGen/<wbr>AArch64/misched-fusion-addr.ll<br>
>><br>
>> AArch64 full bots are broken since yesterday, and among the commits which<br>
>> are involved into the first issue, it seems to me that this one is likely<br>
>> to be responsible of the offset increase and out of range relocation.<br>
>><br>
>> Does it makes sense for you ?<br>
>><br>
>> Bot failure:<br>
>> <a href="http://lab.llvm.org:8011/builders/clang-cmake-aarch64-full/builds/4829" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/<wbr>builders/clang-cmake-aarch64-<wbr>full/builds/4829</a><br>
>> logs:<br>
>><br>
>> <a href="http://lab.llvm.org:8011/builders/clang-cmake-aarch64-full/builds/4829/steps/ninja%20check%202/logs/stdio" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/<wbr>builders/clang-cmake-aarch64-<wbr>full/builds/4829/steps/ninja%<wbr>20check%202/logs/stdio</a><br>
>><br>
>><br>
>> Cheers,<br>
>> Yvan<br>
>><br>
>> > ______________________________<wbr>_________________<br>
>> > llvm-commits mailing list<br>
>> > <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
>> > <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-commits</a><br>
>> ><br>
><br>
><br>
><br>
><br>
> --<br>
> --<br>
> Peter<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">-- <div>Peter</div></div></div>
</div>