[Openmp-commits] [PATCH] D135162: [OPENMP] New api ompx_get_team_procs(devid)

[Shilei Tian via Phabricator via Openmp-commits]

tianshilei1992 added a comment.

The new interface to query the number of physical processors (or whatever it should be called) is fine, but the name is a little bit questionable. It implies how an OpenMP team is mapped to the low level model. I think it is possible that the execution model mapping is per-kernel. It doesn't sound great to have that implication in a general interface name.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D135162/new/

https://reviews.llvm.org/D135162