[PATCH] D32191: Enable lsan by default on 64-bit darwin, and enable lsan test suite

Aleksey Shlyapnikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 20 14:05:24 PDT 2017


alekseyshl accepted this revision.
alekseyshl added inline comments.
This revision is now accepted and ready to land.


================
Comment at: test/lsan/lit.common.cfg:73
+   and (config.host_os is not 'Darwin' or config.target_arch is not 'x86_64'):
   config.unsupported = True
 
----------------
Too many negatives :) maybe break it up?

  supported_linux = config.host_os is 'Linux' and config.host_arch in ['x86_64', 'mips64', 'arm', 'armhf', 'armv7l']
  supported_darwin = config.host_os is 'Darwin' and config.target_arch is 'x86_64'
  if not (supported_linux or supported_darwin):
    config.unsupported = True


https://reviews.llvm.org/D32191





More information about the llvm-commits mailing list