Fix compiler warnings in OpenMP examples with ENABLE_WARNINGS=ON (#562)

LLNL · Jun 27, 2024 · ad9091f · ad9091f
1 parent 87d3861
commit ad9091f
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 16 deletions.
diff --git a/src/examples/openmp-examples/variorum-print-power-limit-openmp-example.c b/src/examples/openmp-examples/variorum-print-power-limit-openmp-example.c
@@ -11,8 +11,7 @@
 
 int main(int argc, char **argv)
 {
-    int ret;
-    int tid;
+    int ret = 0;
 
     const char *usage = "Usage: %s [-h] [-v]\n";
     int opt;
@@ -32,9 +31,9 @@ int main(int argc, char **argv)
         }
     }
 
-    #pragma omp parallel private(tid)
+    #pragma omp parallel
     {
-        tid = omp_get_thread_num();
+        int tid = omp_get_thread_num();
 
         // higher-level software must check for thread and process safety
         // we assume thread 0 is responsible for monitor and control

diff --git a/src/examples/openmp-examples/variorum-print-power-openmp-example.c b/src/examples/openmp-examples/variorum-print-power-openmp-example.c
@@ -11,8 +11,7 @@
 
 int main(int argc, char **argv)
 {
-    int ret;
-    int tid;
+    int ret = 0;
 
     const char *usage = "Usage: %s [-h] [-v]\n";
     int opt;
@@ -32,9 +31,9 @@ int main(int argc, char **argv)
         }
     }
 
-    #pragma omp parallel private(tid)
+    #pragma omp parallel
     {
-        tid = omp_get_thread_num();
+        int tid = omp_get_thread_num();
 
         // higher-level software must check for thread and process safety
         // we assume thread 0 is responsible for monitor and control

diff --git a/src/examples/openmp-examples/variorum-print-verbose-power-limit-openmp-example.c b/src/examples/openmp-examples/variorum-print-verbose-power-limit-openmp-example.c
@@ -11,8 +11,7 @@
 
 int main(int argc, char **argv)
 {
-    int ret;
-    int tid;
+    int ret = 0;
 
     const char *usage = "Usage: %s [-h] [-v]\n";
     int opt;
@@ -32,9 +31,9 @@ int main(int argc, char **argv)
         }
     }
 
-    #pragma omp parallel private(tid)
+    #pragma omp parallel
     {
-        tid = omp_get_thread_num();
+        int tid = omp_get_thread_num();
 
         // higher-level software must check for thread and process safety
         // we assume thread 0 is responsible for monitor and control

diff --git a/src/examples/openmp-examples/variorum-print-verbose-power-openmp-example.c b/src/examples/openmp-examples/variorum-print-verbose-power-openmp-example.c
@@ -26,8 +26,7 @@ static inline double do_work(int input)
 
 int main(int argc, char **argv)
 {
-    int ret;
-    int tid;
+    int ret = 0;
 #ifdef SECOND_RUN
     int i;
     int size = 1E3;
@@ -52,9 +51,9 @@ int main(int argc, char **argv)
         }
     }
 
-    #pragma omp parallel private(tid)
+    #pragma omp parallel
     {
-        tid = omp_get_thread_num();
+        int tid = omp_get_thread_num();
 
         // higher-level software must check for thread and process safety
         // we assume thread 0 is responsible for monitor and control