aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
LGTM, with one formatting nit.
================
Comment at: lib/Sema/SemaDeclAttr.cpp:6201
+void Sema::ProcessDeclAttributeDelayed(Decl *D, const AttributeList *AttrList)
{
+ for (const AttributeList* l = AttrList; l; l = l->getNext())
+ if (l->getKind() == AttributeList::AT_TransparentUnion) {
----------------
aaron.ballman wrote:
> Nit: the `*` should bind to `l` and `l` should be `L` by our usual naming
> conventions (but a name better than `L` wouldn't be amiss).
Nit: missing a space before the first `=`. You should run clang-format on the
patch to catch this sort of thing.
https://reviews.llvm.org/D28266
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
