kpuatamazon commented on issue #19502: URL: https://github.com/apache/incubator-mxnet/issues/19502#issuecomment-724176697
The Centos 7 OpenMP performance is terrible. To test @leezu 's hypothesis, I created a branch https://github.com/kpuatamazon/incubator-mxnet/tree/removeopenmp without the 3rdparty OpenMP. This means there should only be the OS-/compiler-provided OpenMP. The same test above took 542.95s with intgemm's CMakeLists.txt as is, and 539.99s with my OpenMP commented out like below. No real difference here. ``` #option(USE_OPENMP "Use OpenMP" OFF) #if (USE_OPENMP) # message(STATUS "Compiling with OpenMP") # find_package(OpenMP) # if (NOT ${OpenMP_CXX_FOUND}) # message(SEND_ERROR "OpenMP requested but C++ support not found") # endif() # add_compile_options(${OpenMP_CXX_FLAGS}) # target_link_libraries(intgemm PUBLIC OpenMP::OpenMP_CXX) #endif() ``` ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
