lebeg opened a new pull request #12160: Remove conflicting llvm OpenMP from 
cmake builds
URL: https://github.com/apache/incubator-mxnet/pull/12160
 
 
   ## Description ##
   
   Using a different OpenMP library from the one supplied by the compiler can 
lead to undefined behaviour. Multiple problems were reported by users of MXNet.
   
   If there is need to use OpenMP library provided by llvm, one should compile 
MXNet with latest Clang, which is now supported as a test stage by the CI.
   
   ## Checklist ##
   ### Essentials ###
   - [x] This is a minor change
   - [x] Changes are complete
   
   ### Changes ###
   - [x] Removed OpenMP overwrite from CMakeLists.txt
   - [x] Removed otherwise not used OpenMP submodule
   
   ### Comments ###
   Fixes:
   * https://github.com/apache/incubator-mxnet/issues/11417 libomp.so 
dependency (need REAL fix)
   * https://github.com/apache/incubator-mxnet/issues/10856 
test_multi_worker_forked_data_loader fails with DEBUG=1

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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