Update the wording of the bulk_execution_affinity properties in P1436

codeplaysoftware / standards-proposals

Repository for publicly sharing proposals in various standards groups

Apache License 2.0

27 stars 17 forks source link

Update the wording of the bulk_execution_affinity properties in P1436 #112

Closed AerialMantis closed 4 years ago

AerialMantis commented 4 years ago

Rename bulk_execution_affinity_t::scatter_t to bulk_execution_affinity_t::spread_t.
Rename bulk_execution_affinity_t::compact_t to bulk_execution_affinity_t::close_t.
Refine the wording of the bulk_execution_affinity_t properties to clarify the requirements on binding and chunking, based on feedback from SG1.

AerialMantis commented 4 years ago

@mhoemmen thanks for the feedback.

I have updated the MR to apply the suggested changes, and also added some additional wording to describe the terms of art used in the definition of the bulk_execution_affinity_t properties and a note to state that we intend to propose a follow-up property that specifies how larger iteration spaces are subdivided into chunks.

mhoemmen commented 4 years ago

@AerialMantis Thanks! :-D Sorry I've been quiet lately -- I've been working on P1673 pretty hard :-) .