[PATCH] D52434: [OpenMP] Make default schedules for NVPTX target regions in SPMD mode achieve coalescing

Gheorghe-Teodor Bercea via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 27 08:20:07 PDT 2018


gtbercea added a comment.

In https://reviews.llvm.org/D52434#1248012, @Hahnfeld wrote:

> Should we also change the default `schedule` to `static, 1`? I know that's not really needed for `teams distribute parallel for` (because the new default `dist_schedule` only leaves one iteration per thread), but this doesn't happen for `target parallel for`. Additionally it would make the intent more explicit and LLVM doesn't need to look through divisions needed to implement `static` without chunk. Just thinking aloud, not sure if that's worth it.


That is the intention. I just took out that part from here to not confuse things since I wanted to have that as a separate patch. :)


Repository:
  rC Clang

https://reviews.llvm.org/D52434





More information about the cfe-commits mailing list