bgawrych opened a new pull request #20367: URL: https://github.com/apache/incubator-mxnet/pull/20367
## Description ## This PR fixes slowdown on CI (https://github.com/apache/incubator-mxnet/issues/20092) introduced after removing 3rdparty/openmp. GOMP by default doesn't use OMP_DYNAMIC and OMP_NESTED which allows to run parallel regions inside other parallel region. e.g. ``` #pragma omp parallel for for(int i .....) { do_sth()... #pragma omp parallel for for(int i ....) {} } ``` Even if there is "**if**" directive causing sequential run it's still treated as parallel region - setting OMP_DYNAMIC and OMP_NESTED somewhat helps, but seems that without "**if**" clause in GOMP we get even better performance and we don't need to set above env variables. CC: @akarbown ## Checklist ## ### Essentials ### - [x] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc) - [x] Changes are complete (i.e. I finished coding on this PR) - [ ] All changes have test coverage - [ ] Code is well-documented -- 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]
