[Openmp-commits] [PATCH] D14055: [OPENMP] Add dependency to clang/clang-headers etc. for in-tree build of libomp.

[Chandler Carruth via Openmp-commits]

chandlerc added inline comments.

================
Comment at: runtime/test/CMakeLists.txt:57
@@ -56,3 +56,3 @@
   endif()
-  set(LIBOMP_TEST_OPENMP_FLAG -fopenmp=libomp)
+  set(LIBOMP_TEST_OPENMP_FLAG -fopenmp)
   # Use add_lit_testsuite() from LLVM CMake.
----------------
This seems like it should be landed separately, and after your patch to Clang lands?

================
Comment at: runtime/test/api/omp_get_wtime.c:12
@@ -11,3 +11,3 @@
   double measured_time;
-  double wait_time = 0.25; 
+  double wait_time = 1.0; 
   start = 0;
----------------
The commit log doesn't make it clear why this has changed?


http://reviews.llvm.org/D14055