sammccall added a comment.

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.


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

Reply via email to