[Openmp-commits] [PATCH] D36510: OMP_PROC_BIND: better spread

Terry Wilmarth via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Thu Aug 10 07:45:22 PDT 2017


tlwilmar added a comment.

So I think in summary, what you are doing is creating the place partitions with the ceil(P/T)-sized partitions evenly spread amongst the floor(P/T) partitions (where P is num places and T is num_threads).  Is that right?

This looks good, and I don't see any problem with this wrt OMP spec  Can you add some descriptive comments in the code to make it clearer?

Johnny, does it look okay to you?

Thanks!
Terry


Repository:
  rL LLVM

https://reviews.llvm.org/D36510





More information about the Openmp-commits mailing list