[PATCH] D31739: Add markup for libc++ dylib availability
Mehdi AMINI via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Mar 11 09:14:15 PDT 2019
mehdi_amini marked an inline comment as done.
mehdi_amini added subscribers: ldionne, Bigcheese.
mehdi_amini added inline comments.
================
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
----------------
thakis wrote:
> Did you mean `not in` here? I don't understand why we want to skip this tests is availability annotations are present.
Thanks for looking into this, unfortunately I don't remember the context from 2 years ago. It may have been a typo indeed.
@ldionne and @Bigcheese may be able to help here!
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