[llvm-dev] Removed a call to EmitXRayTable() from ARMAsmPrinter

Dean Michael Berris via llvm-dev llvm-dev at lists.llvm.org
Mon Jan 9 22:05:15 PST 2017


> On 10 Jan 2017, at 17:00, Anton Korobeynikov <anton at korobeynikov.info> wrote:
> 
>> 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. :(
> I believe that for this particular case one does not need to have
> execution tests - you'd simply need to check for expected output and
> that's it.

Oh, that's a good point. Yes, we definitely ought to be looking for the instrumentation map in the codegen tests for arm7.

That's probably something we can set up in the same change, and ought to be simple to do anyway.

-- Dean



More information about the llvm-dev mailing list