[PATCH] D15524: [GCC] Attribute ifunc support in clang

John McCall via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 29 12:26:21 PST 2015


rjmccall added inline comments.

================
Comment at: include/clang/AST/DeclBase.h:563
@@ +562,3 @@
+  /// \brief Return true if this declaration is a definition of alias or ifunc.
+  bool hasDefiningAttr() const;
+
----------------
aaron.ballman wrote:
> I think this function and getDefiningAttr() can be defined in the header instead of split into the source file. The implementations are short enough that inlining may be nice to allow.
getDefiningAttr can't be defined in the header without including Attr.h.  We could define hasDefiningAttr in terms of getDefiningAttr, but it wouldn't allow much interesting optimization.


http://reviews.llvm.org/D15524





More information about the cfe-commits mailing list