[PATCH] D23003: [ObjC Availability] Warn upon unguarded use of partially available declaration

Devin Coughlin via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 2 12:35:25 PDT 2016


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





More information about the cfe-commits mailing list