https://github.com/Xazax-hun approved this pull request.

Overall, looks good to me once the review comments from @usx95 are addressed. I 
am just wondering if we should keep this behind a flag initially. The main 
motivation for that would be the understandability of these diagnostics. These 
are very useful, but it might be hard to follow what is going on without some 
additional diagnostics explaining what is happening between the function calls. 

Emitting some additional notes like "parameter N is inferred to be 
lifetimebound" for all the calls involved might make it easier to follow. 

https://github.com/llvm/llvm-project/pull/171081
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to