erichkeane added a comment.

In D146178#4214972 <https://reviews.llvm.org/D146178#4214972>, 
@alexander-shaposhnikov wrote:

> Add more tests.
> P.S. we already have tests with self-friends (in concepts.cpp), the test from 
> Richard's comment is also included (struct S12) (in a slightly simplified 
> form)

Interesting, that self-friend version did NOT fail for me (see the change I 
made which was much less involved).  I think on this one I'm unable to review 
it without Richard, so hopefully he can come back and give comments.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D146178/new/

https://reviews.llvm.org/D146178

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

Reply via email to