Re: Performance regression from removing libiomp5.so

2019-12-12 Thread Lausen, Leonard
That error should be fixed by Chris's work at https://github.com/apache/incubator-mxnet/pull/17039 It is currently expected that libmxnet.so transitively requires both libomp.so and libgomp.so. If this is an issue, we need to build OpenBLAS from source as part of our build scripts, because it

Re: Performance regression from removing libiomp5.so

2019-12-12 Thread Tao Lv
Hi Chris, >From the licensing standpoint, llvm omp is indeed a choice. But previously we noticed that building mxnet with cmake and the llvm omp under 3rdparty folder will cause two runtimes linked to libmxnet.so [1]. Do you think that's still a problem? Also with the current two build systems,

Re: Performance regression from removing libiomp5.so

2019-12-12 Thread Chris Olivier
Hi Patric, The llvm openmp we compile (originally from same Intel source as we all know) seems to be Apache 2.0 licensed. Could we use that instead from a licensing standpoint? On Wed, Dec 11, 2019 at 10:36 PM Zhao, Patric wrote: > Thanks, Sam. > > The root cause is from different OpenMP