[GitHub] [incubator-mxnet] lebeg opened a new pull request #12160: Remove conflicting llvm OpenMP from cmake builds

[GitBox <gi...@apache.org>]

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 to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services