[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