<div dir="ltr">XRay is tested automatically on build-bots with tests in LLVM and compiler-rt . Or are you asking for manual testing instructions?<div>Of these 2 patches, the compiler-rt patch depends on LLVM patch because the tests compiler-rt\test\xray\TestCases\Linux would fail without the fix in LLVM. The compiler-rt patch also enables the tests which were occasionally disabled.</div><div><br></div><div>The XRay feature is quite isolated, so side-effects are not expected. I can only see a risk that some commit I took for granted when making&testing the fixes, somehow didn't get to 4.0 branch.</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 26 January 2017 at 21:44, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">How is XRay tested? IIRC, Renato didn't see any test failures on ARM?<br>
<br>
Merging sounds reasonbaly, I'd just like to understand what's the risk<br>
for the branch.<br>
<div class="HOEnZb"><div class="h5"><br>
On Thu, Jan 26, 2017 at 10:29 AM, Serge Rogatch <<a href="mailto:serge.rogatch@gmail.com">serge.rogatch@gmail.com</a>> wrote:<br>
> Hans, these changes reached trunk in <a href="https://reviews.llvm.org/rL292516" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>rL292516</a> and<br>
> <a href="https://reviews.llvm.org/rL292517" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>rL292517</a> . Could you look?<br>
><br>
> On 26 January 2017 at 03:29, Serge Rogatch <<a href="mailto:serge.rogatch@gmail.com">serge.rogatch@gmail.com</a>> wrote:<br>
>><br>
>> Sorry, I initially included LLVM-Commits rather than LLVM-Dev. Fixed.<br>
>><br>
>> On 26 January 2017 at 03:26, Serge Rogatch <<a href="mailto:serge.rogatch@gmail.com">serge.rogatch@gmail.com</a>><br>
>> wrote:<br>
>>><br>
>>> Hi Dean, Renato,<br>
>>><br>
>>> AFAIK, unfortunately, these critical Arm32 XRay fixes are not yet in 4.0:<br>
>>> <a href="https://reviews.llvm.org/D28624" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D28624</a> , <a href="https://reviews.llvm.org/D28623" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D28623</a> . The<br>
>>> first repairs XRay instrumentation map emission. Without this map XRay<br>
>>> doesn't work at all: the runtime doesn't see anything to instrument. The<br>
>>> second fixes the CPU cache incoherency problem. Without this patch, XRay<br>
>>> will intermittently fail to patch or unpatch some sleds (depending on<br>
>>> whether their code is in CPU cache or not).<br>
>>><br>
>>> Is there any chance we can get these patches to 4.0? This page<br>
>>> <a href="http://llvm.org/docs/HowToReleaseLLVM.html#release-patch-rules" rel="noreferrer" target="_blank">http://llvm.org/docs/<wbr>HowToReleaseLLVM.html#release-<wbr>patch-rules</a> says that<br>
>>> "release manager, the official release testers or the code owners" may<br>
>>> approve cherry-picking to release branch from trunk. AFAIK, you are code<br>
>>> owners for XRay and ARM. I don't know who are the release manager or<br>
>>> official release testers.<br>
>>><br>
>>> Cheers,<br>
>>> Serge<br>
>><br>
>><br>
><br>
</div></div></blockquote></div><br></div>