[PATCH] D42924: Don't pass ForDefinition_t in places it is redundant.
Reid Kleckner via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Feb 7 10:35:29 PST 2018
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
More information about the cfe-commits
mailing list