llvm dev - May 2015 - [LLVMdev] [Openmp-dev] OpenMP 3.1 Implementation Complete

Jack,

Alexey [Bataev] promised to send it for review in a day or two. Then it should
be approved by code reviewers, which might take some time.

andrey

Отправлено с iPad
> 12 мая 2015 г., в 21:22, Jack Howarth <howarth.mailing.lists at
gmail.com> написал(а):
> 
> Andrey,
>     Any idea when the patch to enable openmp as the default for
> -fopenmp will be posted to cfe-commits?
>                Jack
> 
> On Fri, May 8, 2015 at 4:10 PM, Andrey Bokhanko
> <andreybokhanko at gmail.com> wrote:
>> No, just changing defaults -- subject to code reviewers approval.
>> 
>> As I said before, I prefer to leave library naming to library pros.
>> 
>> Andrey
>> 
>> 
>> On Fri, May 8, 2015 at 7:21 PM, Jack Howarth
>> <howarth.mailing.lists at gmail.com> wrote:
>>> So you plan on switching and enabling the openmp library defaults
as
>>> well as changing the openmp library name at the same time?
>>>        Jack
>>> 
>>> On Fri, May 8, 2015 at 12:04 PM, andreybokhanko
>>> <andreybokhanko at gmail.com> wrote:
>>>> It will come on next week.
>>>> 
>>>> I'd like to give a chance to everyone to raise their
objections first.
>>>> 
>>>> Yours,
>>>> Andrey
>>>> 
>>>>> 8 мая 2015 г., в 18:30, Jack Howarth
<howarth.mailing.lists at gmail.com> написал(а):
>>>>> 
>>>>> Is there a proposed patch yet for switching the -fopenmp
support over
>>>>> to the openmp library instead of libgomp? I realize the
call to rename
>>>>> the library may be complicating this process. Perhaps the
switch of
>>>>> the default library should be done first under the current
libiomp5
>>>>> name and that tested for a few days first before using
follow on
>>>>> patches to switch the library name to llomp or whatever. I
would be
>>>>> nice to no longer have to feed clang
"-fopenmp=libiomp5 -Xclang
>>>>> -fopenmp=libiomp5".
>>>>>          Jack
>>>>> 
>>>>> On Fri, May 8, 2015 at 9:26 AM, Andrey Bokhanko
>>>>> <andreybokhanko at gmail.com> wrote:
>>>>>> On Thu, May 7, 2015 at 10:53 PM, Hans Wennborg <hans
at chromium.org> wrote:
>>>>>>> Congratulations! Would you like to add a blurb to
the release notes
>>>>>>> for the next release?
>>>>>> 
>>>>>> Sure! Will do.
>>>>>> 
>>>>>> Yours,
>>>>>> Andrey
>>>>>> _______________________________________________
>>>>>> Openmp-dev mailing list
>>>>>> Openmp-dev at dcs-maillist2.engr.illinois.edu
>>>>>> http://lists.cs.uiuc.edu/mailman/listinfo/openmp-dev

On Tue, May 12, 2015 at 3:58 PM,  <andreybokhanko at gmail.com>
wrote:
> Jack,
>
> Alexey [Bataev] promised to send it for review in a day or two. Then it
should be approved by code reviewers, which might take some time.
>
> andrey
Andrey,
       There is one issue with the proposed patch at
http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20150511/129075.html
with regard to finding the installed libiomp5.dylib for linkage. If
the cmake build of llvm/clang/opemmp uses
-DCMAKE_INSTALL_PREFIX=/sw/opt/llvm-3.7, the -fopenmp option fails to
find the libiomp5.dylib installed in /sw/opt/llvm-3.7/lib for
linkage...

% clang-3.7 -fopenmp -o omp_getEnvInfo omp_getEnvInfo.c -v
clang version 3.7.0 (trunk)
Target: x86_64-apple-darwin14.4.0
Thread model: posix
 "/sw/opt/llvm-3.7.0/bin/clang-3.7" -cc1 -triple
x86_64-apple-macosx10.10.0 -emit-obj -mrelax-all -disable-free
-disable-llvm-verifier -main-file-name omp_getEnvInfo.c
-mrelocation-model pic -pic-level 2 -mthread-model posix
-mdisable-fp-elim -masm-verbose -munwind-tables -target-cpu core2
-target-linker-version 242 -v -dwarf-column-info
-fno-unique-section-names -resource-dir
/sw/opt/llvm-3.7.0/bin/../lib/clang/3.7.0 -fdebug-compilation-dir
/Users/howarth/openmp_examples -ferror-limit 19 -fmessage-length 140
-fopenmp -stack-protector 1 -mstackrealign -fblocks
-fobjc-runtime=macosx-10.10.0 -fencode-extended-block-signature
-fmax-type-align=16 -fdiagnostics-show-option -fcolor-diagnostics -o
/var/folders/1l/n78sywl52lz6kkys6nv7mnph0000gp/T/omp_getEnvInfo-e20595.o
-x c omp_getEnvInfo.c
clang -cc1 version 3.7.0 based upon LLVM 3.7.0svn default target
x86_64-apple-darwin14.4.0
ignoring nonexistent directory "/usr/local/include"
#include "..." search starts here:
#include <...> search starts here:
 /sw/opt/llvm-3.7.0/bin/../lib/clang/3.7.0/include
 /usr/include
 /System/Library/Frameworks (framework directory)
 /Library/Frameworks (framework directory)
End of search list.
 "/sw/opt/llvm-3.7.0/bin/ld" -demangle -dynamic -arch x86_64
-macosx_version_min 10.10.0 -o omp_getEnvInfo -liomp5
/var/folders/1l/n78sywl52lz6kkys6nv7mnph0000gp/T/omp_getEnvInfo-e20595.o
-lSystem
ld: library not found for -liomp5
clang-3.7: error: linker command failed with exit code 1 (use -v to
see invocation)

The location of the installed libiomp5.dylib should always be in the
default library search path for the clang compilers.
                 Jack
>
> Отправлено с iPad
>
>> 12 мая 2015 г., в 21:22, Jack Howarth <howarth.mailing.lists at
gmail.com> написал(а):
>>
>> Andrey,
>>     Any idea when the patch to enable openmp as the default for
>> -fopenmp will be posted to cfe-commits?
>>                Jack
>>
>> On Fri, May 8, 2015 at 4:10 PM, Andrey Bokhanko
>> <andreybokhanko at gmail.com> wrote:
>>> No, just changing defaults -- subject to code reviewers approval.
>>>
>>> As I said before, I prefer to leave library naming to library pros.
>>>
>>> Andrey
>>>
>>>
>>> On Fri, May 8, 2015 at 7:21 PM, Jack Howarth
>>> <howarth.mailing.lists at gmail.com> wrote:
>>>> So you plan on switching and enabling the openmp library
defaults as
>>>> well as changing the openmp library name at the same time?
>>>>        Jack
>>>>
>>>> On Fri, May 8, 2015 at 12:04 PM, andreybokhanko
>>>> <andreybokhanko at gmail.com> wrote:
>>>>> It will come on next week.
>>>>>
>>>>> I'd like to give a chance to everyone to raise their
objections first.
>>>>>
>>>>> Yours,
>>>>> Andrey
>>>>>
>>>>>> 8 мая 2015 г., в 18:30, Jack Howarth
<howarth.mailing.lists at gmail.com> написал(а):
>>>>>>
>>>>>> Is there a proposed patch yet for switching the
-fopenmp support over
>>>>>> to the openmp library instead of libgomp? I realize the
call to rename
>>>>>> the library may be complicating this process. Perhaps
the switch of
>>>>>> the default library should be done first under the
current libiomp5
>>>>>> name and that tested for a few days first before using
follow on
>>>>>> patches to switch the library name to llomp or
whatever. I would be
>>>>>> nice to no longer have to feed clang
"-fopenmp=libiomp5 -Xclang
>>>>>> -fopenmp=libiomp5".
>>>>>>          Jack
>>>>>>
>>>>>> On Fri, May 8, 2015 at 9:26 AM, Andrey Bokhanko
>>>>>> <andreybokhanko at gmail.com> wrote:
>>>>>>> On Thu, May 7, 2015 at 10:53 PM, Hans Wennborg
<hans at chromium.org> wrote:
>>>>>>>> Congratulations! Would you like to add a blurb
to the release notes
>>>>>>>> for the next release?
>>>>>>>
>>>>>>> Sure! Will do.
>>>>>>>
>>>>>>> Yours,
>>>>>>> Andrey
>>>>>>> _______________________________________________
>>>>>>> Openmp-dev mailing list
>>>>>>> Openmp-dev at dcs-maillist2.engr.illinois.edu
>>>>>>>
http://lists.cs.uiuc.edu/mailman/listinfo/openmp-dev

Jack, this is not a problem of this patch, this a problem of your 
configuration. This patch uses standard clang machinery for locating 
libiomp5 library.

Best regards,
Alexey Bataev
============Software Engineer
Intel Compiler Team

13.05.2015 15:59, Jack Howarth пишет:
> On Tue, May 12, 2015 at 3:58 PM,  <andreybokhanko at gmail.com>
wrote:
>> Jack,
>>
>> Alexey [Bataev] promised to send it for review in a day or two. Then it
should be approved by code reviewers, which might take some time.
>>
>> andrey
> Andrey,
>         There is one issue with the proposed patch at
>
http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20150511/129075.html
> with regard to finding the installed libiomp5.dylib for linkage. If
> the cmake build of llvm/clang/opemmp uses
> -DCMAKE_INSTALL_PREFIX=/sw/opt/llvm-3.7, the -fopenmp option fails to
> find the libiomp5.dylib installed in /sw/opt/llvm-3.7/lib for
> linkage...
>
> % clang-3.7 -fopenmp -o omp_getEnvInfo omp_getEnvInfo.c -v
> clang version 3.7.0 (trunk)
> Target: x86_64-apple-darwin14.4.0
> Thread model: posix
>   "/sw/opt/llvm-3.7.0/bin/clang-3.7" -cc1 -triple
> x86_64-apple-macosx10.10.0 -emit-obj -mrelax-all -disable-free
> -disable-llvm-verifier -main-file-name omp_getEnvInfo.c
> -mrelocation-model pic -pic-level 2 -mthread-model posix
> -mdisable-fp-elim -masm-verbose -munwind-tables -target-cpu core2
> -target-linker-version 242 -v -dwarf-column-info
> -fno-unique-section-names -resource-dir
> /sw/opt/llvm-3.7.0/bin/../lib/clang/3.7.0 -fdebug-compilation-dir
> /Users/howarth/openmp_examples -ferror-limit 19 -fmessage-length 140
> -fopenmp -stack-protector 1 -mstackrealign -fblocks
> -fobjc-runtime=macosx-10.10.0 -fencode-extended-block-signature
> -fmax-type-align=16 -fdiagnostics-show-option -fcolor-diagnostics -o
> /var/folders/1l/n78sywl52lz6kkys6nv7mnph0000gp/T/omp_getEnvInfo-e20595.o
> -x c omp_getEnvInfo.c
> clang -cc1 version 3.7.0 based upon LLVM 3.7.0svn default target
> x86_64-apple-darwin14.4.0
> ignoring nonexistent directory "/usr/local/include"
> #include "..." search starts here:
> #include <...> search starts here:
>   /sw/opt/llvm-3.7.0/bin/../lib/clang/3.7.0/include
>   /usr/include
>   /System/Library/Frameworks (framework directory)
>   /Library/Frameworks (framework directory)
> End of search list.
>   "/sw/opt/llvm-3.7.0/bin/ld" -demangle -dynamic -arch x86_64
> -macosx_version_min 10.10.0 -o omp_getEnvInfo -liomp5
> /var/folders/1l/n78sywl52lz6kkys6nv7mnph0000gp/T/omp_getEnvInfo-e20595.o
> -lSystem
> ld: library not found for -liomp5
> clang-3.7: error: linker command failed with exit code 1 (use -v to
> see invocation)
>
> The location of the installed libiomp5.dylib should always be in the
> default library search path for the clang compilers.
>                   Jack
>
>> Отправлено с iPad
>>
>>> 12 мая 2015 г., в 21:22, Jack Howarth <howarth.mailing.lists at
gmail.com> написал(а):
>>>
>>> Andrey,
>>>      Any idea when the patch to enable openmp as the default for
>>> -fopenmp will be posted to cfe-commits?
>>>                 Jack
>>>
>>> On Fri, May 8, 2015 at 4:10 PM, Andrey Bokhanko
>>> <andreybokhanko at gmail.com> wrote:
>>>> No, just changing defaults -- subject to code reviewers
approval.
>>>>
>>>> As I said before, I prefer to leave library naming to library
pros.
>>>>
>>>> Andrey
>>>>
>>>>
>>>> On Fri, May 8, 2015 at 7:21 PM, Jack Howarth
>>>> <howarth.mailing.lists at gmail.com> wrote:
>>>>> So you plan on switching and enabling the openmp library
defaults as
>>>>> well as changing the openmp library name at the same time?
>>>>>         Jack
>>>>>
>>>>> On Fri, May 8, 2015 at 12:04 PM, andreybokhanko
>>>>> <andreybokhanko at gmail.com> wrote:
>>>>>> It will come on next week.
>>>>>>
>>>>>> I'd like to give a chance to everyone to raise
their objections first.
>>>>>>
>>>>>> Yours,
>>>>>> Andrey
>>>>>>
>>>>>>> 8 мая 2015 г., в 18:30, Jack Howarth
<howarth.mailing.lists at gmail.com> написал(а):
>>>>>>>
>>>>>>> Is there a proposed patch yet for switching the
-fopenmp support over
>>>>>>> to the openmp library instead of libgomp? I realize
the call to rename
>>>>>>> the library may be complicating this process.
Perhaps the switch of
>>>>>>> the default library should be done first under the
current libiomp5
>>>>>>> name and that tested for a few days first before
using follow on
>>>>>>> patches to switch the library name to llomp or
whatever. I would be
>>>>>>> nice to no longer have to feed clang
"-fopenmp=libiomp5 -Xclang
>>>>>>> -fopenmp=libiomp5".
>>>>>>>           Jack
>>>>>>>
>>>>>>> On Fri, May 8, 2015 at 9:26 AM, Andrey Bokhanko
>>>>>>> <andreybokhanko at gmail.com> wrote:
>>>>>>>> On Thu, May 7, 2015 at 10:53 PM, Hans Wennborg
<hans at chromium.org> wrote:
>>>>>>>>> Congratulations! Would you like to add a
blurb to the release notes
>>>>>>>>> for the next release?
>>>>>>>> Sure! Will do.
>>>>>>>>
>>>>>>>> Yours,
>>>>>>>> Andrey
>>>>>>>> _______________________________________________
>>>>>>>> Openmp-dev mailing list
>>>>>>>> Openmp-dev at dcs-maillist2.engr.illinois.edu
>>>>>>>>
http://lists.cs.uiuc.edu/mailman/listinfo/openmp-dev