[PATCH] D31739: Add markup for libc++ dylib availability

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 11 08:14:48 PDT 2019


thakis added inline comments.
Herald added subscribers: llvm-commits, jdoerfert, jfb, christof.
Herald added a reviewer: serge-sans-paille.
Herald added a project: LLVM.


================
Comment at: libcxx/trunk/test/std/thread/thread.mutex/thread.lock/thread.lock.shared/lit.local.cfg:1
+if 'availability' in config.available_features:
+    config.unsupported = True
----------------
Did you mean `not in` here? I don't understand why we want to skip this tests is availability annotations are present.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D31739/new/

https://reviews.llvm.org/D31739





More information about the llvm-commits mailing list