akarbown commented on pull request #20474:
URL: https://github.com/apache/incubator-mxnet/pull/20474#issuecomment-898455853


   > Hi @akarbown It looks really good. Thanks! I have a question.
   > 
   > Basically, there are two ways: either we'll use MKL_THREADING_GNU or 
MKL_THREADING_INTEL. Could you please tell me whether there are differences in 
performance results, or, there is no impact on that?
   
   @mozga-intel, this boils down to comparison between ICX and GCC compilers. 
The whole point of this PR is mainly to enable using dynamic linking of MKL 
libraries without hang or different OpenMP symbol conflicts. Thus performance 
of this 2 compilers is not really an issue here. Unit tests for both compilers 
run more or less in the same time (on my local machine). I've observed 10 more 
tests failures for icx than for gcc (unittest_ubuntu_python3_cpu), but it seems 
to be because of the fact that it wasn't tested before and there were problem 
with compiling MxNET with icc at all.


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to