[Openmp-commits] [PATCH] D50084: [test] Convert test for PR36720 to c89

Jonas Hahnfeld via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Tue Jul 31 10:52:22 PDT 2018


Hahnfeld created this revision.
Hahnfeld added reviewers: jlpeyton, AndreyChurbanov.
Herald added a subscriber: openmp-commits.

GCC 4.8.5 defaults to this old C standard. I think we should make the
tests pass a newer -std=c99|c11 but that's too intrusive for now...


Repository:
  rOMP OpenMP

https://reviews.llvm.org/D50084

Files:
  runtime/test/tasking/bug_36720.c


Index: runtime/test/tasking/bug_36720.c
===================================================================
--- runtime/test/tasking/bug_36720.c
+++ runtime/test/tasking/bug_36720.c
@@ -16,15 +16,17 @@
 int main() {
   #pragma omp task
   {
+    int i;
     #pragma omp parallel for
-    for (int i = 0; i < N; i++)
+    for (i = 0; i < N; i++)
       (void)0;
   }
 
   #pragma omp task
   {
+    int i;
     #pragma omp parallel for
-    for (int i = 0; i < N; ++i)
+    for (i = 0; i < N; ++i)
       (void)0;
   }
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D50084.158324.patch
Type: text/x-patch
Size: 520 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20180731/9bf7b8ed/attachment.bin>


More information about the Openmp-commits mailing list