[PATCH] D41736: make attribute instantiation instantiate all attributes

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 4 11:34:06 PST 2018


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





More information about the cfe-commits mailing list