flovent wrote:

> LGTM. I tried to address the specific case of tuple structured bindings in my 
> PR: #147410. Your PR handles the case for all structured bindings. It may be 
> a good idea to add tests for tuple (you can lift from my PR) and array 
> structured bindings too.

Thanks for the review and advice, these testcases have now been added.

https://github.com/llvm/llvm-project/pull/144213
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to