[PATCH] D28623: [XRay][Arm] Enable back XRay testing on Arm32

Dean Michael Berris via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 12 15:42:45 PST 2017


dberris added a comment.

Yes, that was me assuming that XRay wouldn't be supported in 32-bit systems. But I'm happy to be proven wrong there. :D



================
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
----------------
I'm not too familiar with the arm triples, but don't we only support xray on arm7? Does that distinction matter?


https://reviews.llvm.org/D28623





More information about the llvm-commits mailing list