[PATCH] D43382: Xray instrumentation / enabling more tests
Kamil Rytarowski via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Feb 16 06:57:18 PST 2018
krytarowski added a comment.
While there, we should rename the `/Linux/` path to something better for portable tests.
================
Comment at: test/xray/lit.cfg:27
else:
+ if config.host_os in ['FreeBSD'] and host_arch == 'amd64':
+ host_arch = 'x86_64'
----------------
I recommend to move this check out of the `if-else` about `libdl_flag`.
Please remove `config.host_os in ['FreeBSD'] and` the part, it looks unneeded.
Repository:
rCRT Compiler Runtime
https://reviews.llvm.org/D43382
More information about the llvm-commits
mailing list