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
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
