[PATCH] D28623: [XRay][Arm] Enable back XRay testing on Arm32
Serge Rogatch via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jan 13 03:23:39 PST 2017
rSerge added inline comments.
================
Comment at: test/xray/lit.cfg:35
+ config.unsupported = True
+elif config.host_arch.find('64') == -1 and config.host_arch.find('arm') == -1:
config.unsupported = True
----------------
dberris wrote:
> I'm not too familiar with the arm triples, but don't we only support xray on arm7? Does that distinction matter?
AFAIK, build-bots are currently only armv7 . Furthermore, I understood that tests can only execute on architectures in `ALL_XRAY_SUPPORTED_ARCH` from llvm\projects\compiler-rt\cmake\config-ix.cmake , and this makes me question why 64-bitness is checked here again...
https://reviews.llvm.org/D28623
More information about the llvm-commits
mailing list