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
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits