You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2019/07/22 18:38:35 UTC

[GitHub] [incubator-mxnet] larroy commented on issue #15498: Add -DMXNET_USE_OPENMP to Makefiles so runtime info gets updated accordingly

larroy commented on issue #15498: Add -DMXNET_USE_OPENMP to Makefiles so runtime info gets updated accordingly
URL: https://github.com/apache/incubator-mxnet/pull/15498#issuecomment-513903929
 
 
   This is a bit of a departure of how we are testing since we are not testing depending on specific build flags (other than GPU and CPU which we could remove now thanks to libinfo). In that case we should also do all the permutation builds and check the build flags and also for Make and Cmake. This means adding quite a number of recompilations to test this. I'm not sure is worth the increased cost and trouble. Is it justified for you?

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services