andreybokhanko at gmail.com
2015-May-12 19:58 UTC
[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
Jack Howarth
2015-May-13 12:59 UTC
[LLVMdev] [Openmp-dev] OpenMP 3.1 Implementation Complete
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. > > andreyAndrey, 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
Bataev, Alexey
2015-May-13 13:24 UTC
[LLVMdev] [Openmp-dev] OpenMP 3.1 Implementation Complete
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