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

Jonas Hahnfeld via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 27 08:07:11 PDT 2018


Hahnfeld added a comment.

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.


Repository:
  rC Clang

https://reviews.llvm.org/D52434





More information about the cfe-commits mailing list