aaron.ballman marked 2 inline comments as done. aaron.ballman added inline comments.
================ Comment at: test/SemaCXX/warn-shadow.cpp:236 } + void F(int B); // Ok, declaration; not definition. }; ---------------- steveire wrote: > aaron.ballman wrote: > > lebedev.ri wrote: > > > Can you please also add one function with out-of-line definition? > > Gladly! > Is this redundant now that you've added `G`? Sort of. `G` eventually has a body whereas `F` does not -- one of the gates on this logic is a function named `willHaveBody()`, so a bit of redundancy here seems reasonable. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D55321/new/ https://reviews.llvm.org/D55321 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
