[PATCH] D25131: Fix PR 28885: Fix AST Printer output for inheriting constructor using declarations
Richard Smith via cfe-commits
cfe-commits at lists.llvm.org
Fri Sep 30 15:38:27 PDT 2016
rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
Minor comment, otherwise LGTM.
> DeclPrinter.cpp:1350-1351
> +
> + // PR 28885: Use the correct record name when the using declaration is used
> + // for inheriting constructors.
> + for (const auto *Shadow : D->shadows()) {
Don't list PR numbers here. Your code and its comments should be sufficient to explain what you're doing and why without this.
Repository:
rL LLVM
https://reviews.llvm.org/D25131
More information about the cfe-commits
mailing list