leezu commented on issue #17468: Add -march=native -mtune=native to 
config/config.cmake
URL: https://github.com/apache/incubator-mxnet/pull/17468#issuecomment-580056128
 
 
    `cmake -C ../config.cmake` didn't exist previously, in that sense we don't 
break the default behavior. The config file is a new feature in MXNet 1.7 and 
we're free to set the defaults (IMO).
   
   You are right that there may be people that copy the `config.cmake` file, 
don't read it and run into this issue when copying `libmxnet.so` to a different 
computer.
   
   We can comment out the lines introduced by this PR by default to decrease 
the likelihood of running into the issue you describe. Even thene 
https://github.com/apache/incubator-mxnet/issues/14664 would mean that 
`libmxnet.so` would stop working on CPU without AVX.
   
   Would you recommend to comment out the `-march=native -mtune=native` by 
default?

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


With regards,
Apache Git Services

Reply via email to