This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [og7] vector_length extension part 2: Generalize state propagation and synchronization
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