dcoughlin added inline comments.
================
Comment at: lib/Sema/SemaDeclAttr.cpp:6611
@@ +6610,3 @@
+ if (auto *E = dyn_cast<ObjCAvailabilityCheckExpr>(If->getCond())) {
+ // If we're using the '*' case here, then we cannot emit any warnings for
+ // the 'then' branch.
----------------
Do you still want to suppress warnings in the 'then' branch for '*'? I think
there is potential value in pointing out uses of APIs that are not available on
the minimum deployment target for the '*' case.
https://reviews.llvm.org/D23003
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
