rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.

lgtm

I'm guessing things were structured this way so that a function definition 
could have its visibility set before giving it a body, i.e. it would look like 
a declaration, but the visibility should be "for a definition". If everything 
works I suppose we don't do that anymore.


https://reviews.llvm.org/D42924



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D42924: ... Rafael Avila de Espindola via Phabricator via cfe-commits
    • [PATCH] D42... Reid Kleckner via Phabricator via cfe-commits
    • [PATCH] D42... Rafael Avila de Espindola via Phabricator via cfe-commits

Reply via email to