TaoLv commented on a change in pull request #13775: Fix USE_MKLDNN check in Makefile URL: https://github.com/apache/incubator-mxnet/pull/13775#discussion_r245479149
########## File path: make/config.mk ########## @@ -98,8 +98,10 @@ USE_LIBJPEG_TURBO_PATH = NONE # use openmp for parallelization USE_OPENMP = 1 -# whether use MKL-DNN library -USE_MKLDNN = 0 +# whether use MKL-DNN library: 0 = disabled, 1 = enabled +# if USE_MKLDNN is not defined, MKL-DNN will be enabled by default on x86 Linux. +# you can disable it explicity with USE_MKLDNN = 0 +USE_MKLDNN = Review comment: Since we have different default behavior for different platforms at this time, we cannot have a proper default value for `USE_MKLDNN` here. So I leave it undefined. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on 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
