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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to