[LLVMdev] [Openmp-dev] OpenMP 3.1 Implementation Complete

[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