<div dir="ltr">Jonathan,<div>     The second_intel-cmake.patch seems to have introduced a race condition when using parallel make on darwin. I now find that…</div><div><br></div><div>% cd openmp-3.5.0/runtime</div><div>% mkdir build</div>
<div>% cd build</div><div>% cmake -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++ -Dos=mac -Darch=32 ..</div><div>% make -j 2</div><div><div>-- The C compiler identification is Clang 4.2.0</div><div>-- The CXX compiler identification is Clang 4.2.0</div>
<div>-- Check for working C compiler: /usr/bin/clang</div><div>-- Check for working C compiler: /usr/bin/clang -- works</div><div>-- Detecting C compiler ABI info</div><div>-- Detecting C compiler ABI info - done</div><div>
-- Check for working CXX compiler: /usr/bin/clang++</div><div>-- Check for working CXX compiler: /usr/bin/clang++ -- works</div><div>-- Detecting CXX compiler ABI info</div><div>-- Detecting CXX compiler ABI info - done</div>
<div>-- Found Perl: /usr/bin/perl (found version "5.12.3") </div><div><br></div><div>------------------- CONFIGURATION --------------------------</div><div>Operating System : mac</div><div>Architecture     : 32</div>
<div>Build Type       : release</div><div>OpenMP Version   : 40</div><div>Lib Type         : normal</div><div>Stats-Gathering  : OFF</div><div>Fortran Modules  : OFF</div><div>Build            : development</div><div>------------------------------------------------------------</div>
<div><br></div><div>-- The ASM compiler identification is Clang</div><div>-- Found assembler: /usr/bin/clang</div><div>-- Configuring done</div><div>-- Generating done</div><div>-- Build files have been written to: /Users/howarth/llvm-svn/openmp-3.5.0/runtime/build</div>
<div>[Jacks-MacBook-Pro:openmp-3.5.0/runtime/build] howarth% make -j 2</div><div>Scanning dependencies of target inc</div><div>Scanning dependencies of target needed-headers</div><div>[  2%] [  4%] Generating ../exports/mac_32/include_compat/.dir</div>
<div>Generating kmp_i18n_id.inc</div><div>[  6%] Generating iomp_lib.h</div><div>[  8%] Generating ../exports/mac_32/include_compat/iomp_lib.h</div><div>[  8%] Built target inc</div><div>[ 10%] Generating kmp_i18n_default.inc</div>
<div>[ 12%] Generating omp.h</div><div>Scanning dependencies of target lib</div><div>[ 14%] Generating ../exports/mac_32/lib.thin/.dir</div><div>[ 14%] Built target needed-headers</div><div>make[2]: *** No rule to make target `unstripped/libiomp5.dylib', needed by `libiomp5.dylib'.  Stop.</div>
<div>make[2]: *** Waiting for unfinished jobs....</div><div>Scanning dependencies of target iomp5</div><div>[ 17%] Generating .dir</div><div>make[1]: *** [CMakeFiles/lib.dir/all] Error 2</div><div>make[1]: *** Waiting for unfinished jobs....</div>
<div>[ 19%] [ 21%] Building C object CMakeFiles/iomp5.dir/src/kmp_ftn_cdecl.c.o</div><div>Building C object CMakeFiles/iomp5.dir/src/kmp_ftn_extra.c.o</div><div>[ 23%] [ 25%] Building C object CMakeFiles/iomp5.dir/src/kmp_version.c.o</div>
<div>Building C object CMakeFiles/iomp5.dir/src/kmp_alloc.c.o</div><div>[ 27%] Building C object CMakeFiles/iomp5.dir/src/kmp_atomic.c.o</div><div>[ 29%] Building C object CMakeFiles/iomp5.dir/src/kmp_csupport.c.o</div><div>
[ 31%] Building C object CMakeFiles/iomp5.dir/src/kmp_debug.c.o</div><div>[ 34%] Building C object CMakeFiles/iomp5.dir/src/kmp_itt.c.o</div><div>[ 36%] Building C object CMakeFiles/iomp5.dir/src/kmp_environment.c.o</div>
<div>[ 38%] Building C object CMakeFiles/iomp5.dir/src/kmp_error.c.o</div><div>[ 40%] Building C object CMakeFiles/iomp5.dir/src/kmp_global.c.o</div><div>[ 42%] Building C object CMakeFiles/iomp5.dir/src/kmp_i18n.c.o</div>
<div>[ 44%] Building C object CMakeFiles/iomp5.dir/src/kmp_io.c.o</div><div>[ 46%] Building C object CMakeFiles/iomp5.dir/src/kmp_runtime.c.o</div><div>[ 48%] Building C object CMakeFiles/iomp5.dir/src/kmp_settings.c.o</div>
<div>/Users/howarth/llvm-svn/openmp-3.5.0/runtime/src/kmp_settings.c:3456:29: warning: using the result of an assignment as a condition without</div><div>      parentheses [-Wparentheses]</div><div>            } while ( value = semicolon ? semicolon + 1 : NULL );/Users/howarth/llvm-svn/openmp-3.5.0/runtime/src/kmp_runtime.c:4433:89: warning: for loop has empty body [-Wempty-body]</div>
<div>    for( gtid=(initial_thread ? 0 : 1) ; TCR_PTR(__kmp_threads[gtid]) != NULL ; gtid++ );</div><div>                                                                                        ^</div><div>/Users/howarth/llvm-svn/openmp-3.5.0/runtime/src/kmp_runtime.c:4433:89: note: put the semicolon on a separate line to silence this warning</div>
<div><br></div><div>                      ~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~</div><div>/Users/howarth/llvm-svn/openmp-3.5.0/runtime/src/kmp_settings.c:3456:29: note: place parentheses around the assignment to silence this warning</div>
<div>            } while ( value = semicolon ? semicolon + 1 : NULL );</div><div>                            ^</div><div>                      (                                       )</div><div>/Users/howarth/llvm-svn/openmp-3.5.0/runtime/src/kmp_settings.c:3456:29: note: use '==' to turn this assignment into an equality comparison</div>
<div>            } while ( value = semicolon ? semicolon + 1 : NULL );</div><div>                            ^</div><div>                            ==</div><div>/Users/howarth/llvm-svn/openmp-3.5.0/runtime/src/kmp_settings.c:4018:53: warning: '&&' within '||' [-Wlogical-op-parentheses]</div>
<div>        if ( ( ( *next < '0' ) || ( *next > '9' ) ) && ( *next !=',') || ( total > 2 ) ) {</div><div>             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~ ~~</div>
<div>/Users/howarth/llvm-svn/openmp-3.5.0/runtime/src/kmp_settings.c:4018:53: note: place parentheses around the '&&' expression to silence this</div><div>      warning</div><div>        if ( ( ( *next < '0' ) || ( *next > '9' ) ) && ( *next !=',') || ( total > 2 ) ) {</div>
<div>                                                    ^</div><div>             (                                                       )</div><div>/Users/howarth/llvm-svn/openmp-3.5.0/runtime/src/kmp_settings.c:5167:13: warning: '&&' within '||' [-Wlogical-op-parentheses]</div>
<div>            &&</div><div>            ^~</div><div>/Users/howarth/llvm-svn/openmp-3.5.0/runtime/src/kmp_settings.c:5167:13: note: place parentheses around the '&&' expression to silence this</div>
<div>      warning</div><div>            &&</div><div>            ^</div><div>3 warnings generated.</div><div>[ 51%] Building C object CMakeFiles/iomp5.dir/src/kmp_str.c.o</div><div>1 warning generated.</div><div>
[ 53%] Building C object CMakeFiles/iomp5.dir/src/kmp_tasking.c.o</div><div>[ 55%] Building C object CMakeFiles/iomp5.dir/src/kmp_taskq.c.o</div><div>[ 57%] [ 59%] Building C object CMakeFiles/iomp5.dir/src/kmp_utility.c.o</div>
<div>Building C object CMakeFiles/iomp5.dir/src/kmp_threadprivate.c.o</div><div>[ 61%] Building C object CMakeFiles/iomp5.dir/src/thirdparty/ittnotify/ittnotify_static.c.o</div><div>[ 63%] Building C object CMakeFiles/iomp5.dir/src/z_Linux_util.c.o</div>
<div>[ 65%] Building C object CMakeFiles/iomp5.dir/src/kmp_gsupport.c.o</div><div>[ 68%] Building CXX object CMakeFiles/iomp5.dir/src/kmp_affinity.cpp.o</div><div>[ 70%] Building CXX object CMakeFiles/iomp5.dir/src/kmp_dispatch.cpp.o</div>
<div>[ 72%] Building CXX object CMakeFiles/iomp5.dir/src/kmp_lock.cpp.o</div><div>[ 74%] Building CXX object CMakeFiles/iomp5.dir/src/kmp_sched.cpp.o</div><div>[ 76%] Building CXX object CMakeFiles/iomp5.dir/src/kmp_taskdeps.cpp.o</div>
<div>[ 78%] Building CXX object CMakeFiles/iomp5.dir/src/kmp_cancel.cpp.o</div><div>[ 80%] Building ASM object CMakeFiles/iomp5.dir/src/z_Linux_asm.s.o</div><div>Linking C shared library unstripped/libiomp5.dylib</div><div>
[ 80%] Built target iomp5</div><div>make: *** [all] Error 2</div></div><div><br></div><div>On the plus side, when using 'make -j 1', the recipe for fat binaries no longer requires 'rm -fr *' in between the 32 and 32e builds.</div>
<div>                  Jack</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 30, 2014 at 11:49 AM, Peyton, Jonathan L <span dir="ltr"><<a href="mailto:jonathan.l.peyton@intel.com" target="_blank">jonathan.l.peyton@intel.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello C. Bergström,<br>
<br>
Here is an initial look at the revised cmake system.  I'm still trying to get some Windows parts settled down and more "CMakey".<br>
The differences from the last one:<br>
1) Does not record a configure timestamp so only cmake v2.8.0 and up is required<br>
2) Uses add_library() command for creating library<br>
3) All compiler specific flags are in cmake/${CMAKE_C_COMPILER_ID}Flags.cmake<br>
     If a particular compiler does not have a corresponding compile flags file, then a warning is issued during configuration that only common flags are going to be used (cmake/CommonFlags.cmake)<br>
4) I moved the micro-tests to a separate file (cmake/MicroTests.cmake) remember these can be turned off via -Dtests=off when calling cmake.<br>
5) Windows builds require MASM compiler so specify -DCMAKE_ASM_MASM_COMPILER=... for this assembler rather than -DCMAKE_ASM_COMPILER=...<br>
<br>
[ within the top level directory that contains: runtime/ , offload/ , testsuite/ , www/ ]<br>
To remove the old patch, type:<br>
patch -p0 -RE < initial_intel_cmake.patch<br>
<br>
[ within the top level directory that contains: runtime/ , offload/ , testsuite/ , www/ ]<br>
To add new patch, type:<br>
patch -p0 < second_intel_cmake.patch<br>
<div class="im HOEnZb"><br>
Johnny<br>
<br>
-----Original Message-----<br>
From: "C. Bergström" [mailto:<a href="mailto:cbergstrom@pathscale.com">cbergstrom@pathscale.com</a>]<br>
</div><div class="im HOEnZb">Sent: Monday, June 30, 2014 6:50 AM<br>
To: Peyton, Jonathan L<br>
</div><div class="im HOEnZb">Cc: Cownie, James H; <a href="mailto:openmp-dev@dcs-maillist2.engr.illinois.edu">openmp-dev@dcs-maillist2.engr.illinois.edu</a><br>
Subject: Re: [Openmp-dev] Initial Intel Cmake Build System Patch<br>
<br>
</div><div class="HOEnZb"><div class="h5">On 06/12/14 01:16 AM, Peyton, Jonathan L wrote:<br>
> C. Bergström,<br>
><br>
> I am working on this now.  I will create:<br>
> common_flags.cmake  ,  icc.cmake  ,  gcc.cmake  ,  clang.cmake<br>
><br>
> I hope to have this done in the next few days.<br>
Hi Johnny,<br>
<br>
It's been a couple weeks and I just wanted to ping this thread. Was Intel planning some work on cmake or should (PathScale) or others feel safe to invest time in it?<br>
<br>
Thanks<br>
</div></div><br>_______________________________________________<br>
Openmp-dev mailing list<br>
<a href="mailto:Openmp-dev@dcs-maillist2.engr.illinois.edu">Openmp-dev@dcs-maillist2.engr.illinois.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/openmp-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/openmp-dev</a><br>
<br></blockquote></div><br></div>