[PATCH] D44100: [ASTImporter] Reorder fields after structure import is finished
Aleksei Sidorin via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Mar 6 01:50:29 PST 2018
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
More information about the cfe-commits
mailing list