On Sat, Jan 06, 2018 at 09:21:59AM +0100, Tom de Vries wrote: > this patch adds the following builtins in C/C++: > - __builtin_goacc_gang_id > - __builtin_goacc_worker_id > - __builtin_goacc_vector_id > - __builtin_goacc_gang_size > - __builtin_goacc_worker_size > - __builtin_goacc_vector_size
I wonder if it wouldn't be better to have just 2 builtins instead of 6, with one argument (required to be constant) - the kind of parallelism you're interested in, to avoid the inflation of the builtins. Jakub