<div dir="ltr">Hi,<div><br></div><div>After investigating the problem more, we have a guess that maybe AArch64 CPU doesn't see the code changes which XRay has done, because the old instructions stay in the instruction cache, even though code modification has been performed in the same thread as the one running the instructions later.</div><div><br></div><div>Is this possible that CPU cache stays so incoherent with the data cache?</div><div><br></div><div>Cheers,</div><div>Serge</div></div><div class="gmail_extra"><br><div class="gmail_quote">On 8 December 2016 at 01:48, Dean Michael Berris <span dir="ltr"><<a href="mailto:dean.berris@gmail.com" target="_blank">dean.berris@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"><div>Thanks Serge!<br><br>Sent from my iPhone</div><div><div class="h5"><div><br>On 8 Dec. 2016, at 9:12 am, x y <<a href="mailto:serge.rogatch@gmail.com" target="_blank">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" target="_blank">https://reviews.llvm.org/<wbr>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>On 5 December 2016 at 17:19, x y <<a href="mailto:serge.rogatch@gmail.com" target="_blank">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><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="m_-7021296856490114949HOEnZb"><font color="#888888"><br>
--renato<br>
</font></span></blockquote></div><br></div>
</div></blockquote></div></div></div></blockquote></div><br></div>