[PATCH] D43382: Xray instrumentation / enabling more tests

Kamil Rytarowski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 16 07:37:58 PST 2018


krytarowski accepted this revision.
krytarowski added a comment.
This revision is now accepted and ready to land.

  $ find . -iname posix
  ./test/asan/TestCases/Posix
  ./test/sanitizer_common/TestCases/Posix

I think `Posix` is a better choice as it's already in use.

Other than that - it looks good to me.



================
Comment at: test/xray/lit.cfg:52
 
 if config.host_os not in ['FreeBSD', 'Linux']:
   config.unsupported = True
----------------
This line already contains `'NetBSD'` in HEAD.


https://reviews.llvm.org/D43382





More information about the llvm-commits mailing list