<div dir="ltr">Then I plan to enable back emission of XRay instrumentation map in Arm32 and testing of Arm32 (about 1 line each, but they may break the builds), and to submit this for review together with the tail call patches.<div><br></div><div>Serge</div></div><div class="gmail_extra"><br><div class="gmail_quote">On 10 January 2017 at 09:05, 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"><span class=""><br>
> On 10 Jan 2017, at 17:00, Anton Korobeynikov <<a href="mailto:anton@korobeynikov.info">anton@korobeynikov.info</a>> wrote:<br>
><br>
>> As Oleg points out, apparently the way we've configured the XRay tests in compiler-rt/... excludes running tests on non-64-bit platforms. arm7 is definitely not 64-bit and therefore means we've not been running those tests. :(<br>
> I believe that for this particular case one does not need to have<br>
> execution tests - you'd simply need to check for expected output and<br>
> that's it.<br>
<br>
</span>Oh, that's a good point. Yes, we definitely ought to be looking for the instrumentation map in the codegen tests for arm7.<br>
<br>
That's probably something we can set up in the same change, and ought to be simple to do anyway.<br>
<span class="HOEnZb"><font color="#888888"><br>
-- Dean<br>
<br>
</font></span></blockquote></div><br></div>