a.sidorin added a comment.

Hi Gabor,

> Just a minor high level note. If https://reviews.llvm.org/D32947 will be 
> accepted once, we will need to reorder friends as well. Or alternatively, we 
> have to omit the check of friends in structural equivalence in 
> https://reviews.llvm.org/D32947.

I'd prefer to land https://reviews.llvm.org/D32947 before the fix for friend 
decls reordering so we will have a subject for testing. And the fix for friend 
decls can become a separate patch. Do you already have such patch or a test for 
the issue?


Repository:
  rC Clang

https://reviews.llvm.org/D44100



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

Reply via email to