aaron.ballman added a comment. I don't suppose there is a chance at test coverage for this change?
================ Comment at: include/clang/AST/Attr.h:142 + /// explicitly provided in the current declaration? + bool isInheritEvenIfAlreadyPresent() const { + return InheritEvenIfAlreadyPresent; ---------------- I'm not too keen on the name -- perhaps `isInheritedEvenIfAlreadyPresent()`? Repository: rC Clang https://reviews.llvm.org/D41736 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits