[PATCH] D79094: [SemaObjC] Warn on visibility attributes on an @implementation

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat May 2 08:27:47 PDT 2020


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

This seems reasonable to me. I agree that this feels like it's something that should be written on the declaration in the header, not just magically appear on the definition, but I'm not an expert in ObjC to know if there are use cases where this may be useful.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D79094/new/

https://reviews.llvm.org/D79094





More information about the cfe-commits mailing list