Karthikdhondi wrote:

> This fix only handles pointer-to-VLA. It should also handle:
> 
> 1. Direct VLA captures: `int arr[n]`    (this might already be working - can 
> you please check?).
> 2. References to VLAs: `int (&ref)[n]` and `int (&&ref)[n]` (as Aaron 
> mentioned).
> 3. Multidimensional cases: `int (*p)[n][m]`
> 
> Please add test for each scenario.

Thanks for the review. I updated the regression coverage to include  Direct VLA 
captures, References to VLAs and Multidimensional cases. 

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

Reply via email to