rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.
Looks good, thanks.
It strikes me that this will still lead to inconsistencies. For example, I
expect this:
struct A { struct B *a, *b; struct B *c, *d; };
... to print as:
struct A {
struct B *a, *b;
struct B *c;
struct B *d;
};
... where the first two are joined because their type owns a declaration of
`struct B`, and the second two are not joined because their type does not own a
declaration (it just has a reference to the already-existing declaration of
`struct B`). One (somewhat hacky) way to address this would be to compare the
starting source locations of a sequence of `DeclaratorDecl`s and group them if
it's the same.
https://reviews.llvm.org/D45465
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
