[Openmp-commits] [PATCH] D13827: Avoid bad conversion for __kmp_sys_max_nth

[Jim Cownie via Openmp-commits]

jcownie added a subscriber: jcownie.
jcownie added a comment.

If you choose this approach, 32Ki seems too small. A current maxed-out SGI is already at 256 sockets which could get you to 256*18*2 = 9216 threads, so over 1/4 of the way to your 32Ki.

You’re not limited to 16bits here, so why not just make this BIG (say 1Gi) which we know is unlikely to be hit in the foreseeable future.

- Jim

James Cownie <james.h.cownie at intel.com>
SSG/DPD/TCAR (Technical Computing, Analyzers and Runtimes)
Tel: +44 117 9071438


http://reviews.llvm.org/D13827