[Openmp-commits] [PATCH] D74092: Changed omp_get_max_threads() implementation to more closely match spec description.

Ethan Stewart via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Wed Feb 5 17:33:27 PST 2020

estewart08 added a comment.

I can definitely add the change to max_threads.c to this review. The CHECK would become 64 due to the fact we are counting all threads now with this proposed change 32 thread_limit + 32 master warp.

  // CHECK: Non-SPMD MaxThreadsL1 = 64

Yes, the test I proposed would be for nvptx only due to the fact that the other tests reside in the nvptx directory and the original max_threads test was checking nvptx values as well. Is the plan to convert all tests so that they support different architectures in the future and move them to common?

  rG LLVM Github Monorepo



More information about the Openmp-commits mailing list