eric-haibin-lin commented on a change in pull request #12502: [NGRAPH] MXNet - nGraph initial integration URL: https://github.com/apache/incubator-mxnet/pull/12502#discussion_r225606324
########## File path: Makefile ########## @@ -74,8 +75,10 @@ endif include $(TPARTYDIR)/mshadow/make/mshadow.mk include $(DMLC_CORE)/make/dmlc.mk -# all tge possible warning tread -WARNFLAGS= -Wall -Wsign-compare +include 3rdparty/ngraph-mxnet-bridge/ngraph.mk Review comment: Does NGraph master use a different version of MKLDNN that's being used by MXNet master? Is it possible to align the mkldnn version so that use_ngraph and use_mkldnn can be turned on together? ---------------------------------------------------------------- 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
