Re: [PATCH][openacc] Fix acc declare for VLAs

2020-10-06 Thread Tobias Burnus
And as spotted by Thomas, Tom's patch also resolved an XFAIL in gcc/testsuite. Committed as r11-3687-ga9802204603616df14ed47d05f1b86f1bd08d8fb after testing it on x86-64-gnu-linux. Tobias On 10/6/20 3:28 PM, Tom de Vries wrote: ... [openacc] Fix acc declare for VLAs gcc/ChangeLog: 2020-10-06

Re: [PATCH][openacc] Fix acc declare for VLAs

2020-10-06 Thread Tobias Burnus
LGTM. Thanks, Tobias On 10/6/20 3:28 PM, Tom de Vries wrote: Hi, Consider test-case test.c, with VLA A: ... int main (void) { int N = 1000; int A[N]; #pragma acc declare copy(A) return 0; } ... compiled using: ... $ gcc test.c -fopenacc -S -fdump-tree-all ... At original, we have

[PATCH][openacc] Fix acc declare for VLAs

2020-10-06 Thread Tom de Vries
Hi, Consider test-case test.c, with VLA A: ... int main (void) { int N = 1000; int A[N]; #pragma acc declare copy(A) return 0; } ... compiled using: ... $ gcc test.c -fopenacc -S -fdump-tree-all ... At original, we have: ... #pragma acc declare map(tofrom:A); ... but at gimple, we have