cjolivier01 commented on issue #17641: OpenMP Error
URL: 
https://github.com/apache/incubator-mxnet/issues/17641#issuecomment-590995601
 
 
   > @TaoLv `-DMKL_USE_STATIC_LIBS=ON` is currently broken on my system and in 
particular doesn't link `iomp` statically.
   > 
   > I'll create a PR to statically link iomp by default if MKL is found. As 
all OMP related symbols will then be resolved at compile time, it doesn't 
matter that gcc declares `libgomp.so` and clang `libomp.so` respectively as 
NEEDED. Further I'll disable building llvm openmp when statically linking to 
`libiomp.so`. @cjolivier01 does that sound alright to you?
   
   Linking to omp statically is dangerous because atfork handler registration 
(and thus execution) order is no defined, as well as general global init order, 
of course, although I am not sure if that is problematic since libiomp i all C.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to