kbobyrev added a comment. In D112707#3166703 <https://reviews.llvm.org/D112707#3166703>, @sammccall wrote:
> In D112707#3166535 <https://reviews.llvm.org/D112707#3166535>, @kbobyrev > wrote: > >> @sammccall >> >> There is one part of your comment that I couldn't understand, could you >> please clarify what you mean bi this? >> >>> we can't skip based on a definition (main file or class) that can't stand >>> alone, usually because of qualifiers. e.g. void foo::bar() {}. >> >> FWIW if we decided to only deal with `RecordDecl` I don't know if your >> comment refers to them or it's some other context. An example for what you >> mean would be appreciated! > > Yeah I don't think this applies to classes. > The following is valid but you can't drop the (include of) first declaration: > > namespace a { void foo(); } > void a::foo(); > > However classes cannot be forward-declared with qualifiers in this way. I see, thanks a lot! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D112707/new/ https://reviews.llvm.org/D112707 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
