dcoughlin accepted this revision. dcoughlin added a comment. This revision is now accepted and ready to land.
This looks good to me, but I think we need a deployment target check on the diagnostic since the safe API is only available in iOS 11+, macOS 10.13+, tvOS 11+, and watchOS 4.0+. If the deployment target is early than those versions then we should not diagnose. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71728/new/ https://reviews.llvm.org/D71728 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
