[Openmp-commits] [PATCH] D40949: [OMPT] Add missing ompt_get_num_procs function

Joachim Protze via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Fri Dec 8 04:03:26 PST 2017


protze.joachim added inline comments.


================
Comment at: runtime/test/ompt/misc/api_calls.c:67
 
+  // CHECK: {{^}}[[MASTER_ID]]: omp_get_num_procs()=[[NUM_PROCS:[-]?[0-9]+]]
+  // CHECK: {{^}}[[MASTER_ID]]: ompt_get_num_procs()=[[NUM_PROCS]]
----------------
Hahnfeld wrote:
> Can we get a negative result?!?
The return value is int, thus could be negative.

If a negative value is invalid for omp_get_num_procs, it should be tested somewhere else.


https://reviews.llvm.org/D40949





More information about the Openmp-commits mailing list