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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to