
<html><head><meta http-equiv="content-type" content="text/html; charset=utf-8"></head><body dir="auto"><div>Thanks Serge!<br><br>Sent from my iPhone</div><div><br>On 8 Dec. 2016, at 9:12 am, x y <<a href="mailto:serge.rogatch@gmail.com">serge.rogatch@gmail.com</a>> wrote:<br><br></div><blockquote type="cite"><div><div dir="ltr">The test has been disabled for AArch64 in <a href="https://reviews.llvm.org/rL288988">https://reviews.llvm.org/rL288988</a> .<div><br></div><div>Cheers,</div><div>Serge</div></div><div class="gmail_extra"><br><div class="gmail_quote">On 5 December 2016 at 20:36, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 5 December 2016 at 17:19, x y <<a href="mailto:serge.rogatch@gmail.com">serge.rogatch@gmail.com</a>> wrote:<br>

> I've tried what I could troubleshooting the issue. My AArch64 XRay code is<br>

> quite short, so the only issue I can spot now happens when __arm__ and<br>

> __aarch64__ are defined simultaneously. Shall I submit a fix for better<br>

> handling this case, or should it really never happen?<br>

<br>

</span>Hum, __arm__ should not be defined on AArch64, but I remember having a<br>

similar problem in compiler-rt.<br>

<br>

Can you use #elsif defined(__aarch64__) ?<br>

<span class=""><br>

<br>

> I'm going to disable the test with the approach suggested by Renato. Sorry<br>

> it took so long, I tried to fix the issue first. And of course, I cannot<br>

> reproduce it locally.<br>

<br>

</span>Ok, LGTM. Thanks!<br>

<span class="HOEnZb"><font color="#888888"><br>

--renato<br>

</font></span></blockquote></div><br></div>

</div></blockquote></body></html>