On 03/22/2018 10:39 AM, Tom de Vries wrote: > On 03/02/2018 05:55 PM, Cesar Philippidis wrote: >> + rtx red_partition; /* Similar to bcast_partition, except for vector >> + reductions. */ > > Shouldn't this be in "[og7] vector_length extension part 3: reductions"?
Maybe. But keep in mind, with the exception of the bar.sync and maxntid changes you requested, I don't think the vector length patch makes sense to go in as individual hunks. Maybe I could split out the new TARGET_GOACC_ADJUST_PARALLELISM hook in part 4 into a separate patch. But, at the same time, if something isn't being used, what's the point of going through that extra work? Cesar