HazardyKnusperkeks added a comment. In D113826#3129383 <https://reviews.llvm.org/D113826#3129383>, @lichray wrote:
> Thanks for the patch! It works for me, both for the `auto{x}` and `new > auto{x}` cases. Some corner cases left: > > void f() { T(&a)->n = 1; } > void f() { T{&a}->n = 1; } // xxx > void g() { auto(&a)->n = 0; } > void h() { auto{&a}->n = 0; } // xxx > > The `/// xxx` cases are formatted as > > void f() { > T { &a } -> n = 1; > } > > and > > void h() { > auto{ &a } -> n = 0; > } > > But this may not worth a fix (for now). I'm sure someone will look into it, but this should not be handled by this change. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D113826/new/ https://reviews.llvm.org/D113826 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits