[PATCH] D39913: [ObjC] warn about availability attributes missing from a method's declaration when they're specified for a method's definition

Erik Pilkington via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 17 06:31:12 PST 2017


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

> It's harder as we don't know the distinction between declaration/definition at merge time. Right now the C++ implementation of this warning actually checks the attributes too late, after they're merged. I'll take out the C++/C support completely and keep to ObjC in this patch for simplicity. The rest can be supported later.

That sounds fine for now! LGTM


Repository:
  rL LLVM

https://reviews.llvm.org/D39913





More information about the cfe-commits mailing list