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

[Manman Ren via cfe-commits]

manmanren accepted this revision.
manmanren added a comment.
This revision is now accepted and ready to land.



> This is done so containers, such as `vector<partially_available>` can be used safely provided `partially_available` is safe at the point of instantiation. I think the way to improve this is in `Sema::getVersionForDecl()`. There we could look at function and template parameters to determine the best base version, then diagnose against it.


Let's fix the diagnostics for dependent type in a follow-up patch. And your suggestion above sounds reasonable to me.

Thanks for the work!
Manman


https://reviews.llvm.org/D23003