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

Jonas Hahnfeld via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Thu Dec 7 06:22:34 PST 2017


Hahnfeld accepted this revision.
Hahnfeld added a comment.
This revision is now accepted and ready to land.

LGTM with one change to the test



================
Comment at: runtime/test/ompt/misc/api_calls.c:47-48
+
+    printf("%" PRIu64 ": omp_get_num_procs()=%d\n", ompt_get_thread_data()->value, omp_get_num_procs());
+    printf("%" PRIu64 ": ompt_get_num_procs()=%d\n", ompt_get_thread_data()->value, ompt_get_num_procs());
   }
----------------
Maybe check that these two return the same result as for `omp{,t}_get_num_places`?
```
// CHECK: {{^}}[[MASTER_ID]]: omp_get_num_procs()=[[NUM_PROCS:[0-9]+]]
// CHECK: {{^}}[[MASTER_ID]]: ompt_get_num_procs()=[[NUM_PROCS]]
```


https://reviews.llvm.org/D40949





More information about the Openmp-commits mailing list