Skip to content
This repository was archived by the owner on Nov 17, 2023. It is now read-only.

Conversation

@yuxihu
Copy link
Member

@yuxihu yuxihu commented May 6, 2019

Per changes in #13668, we generate a new header file mkldnn_version.h during MXNet compilation based on the template file mkldnn_version.h.in. This dynamically generated header file needs to be added to the MXNet pip package to prevent compilation failure for building Horovod with MKLDNN enabled MXNet pip package.

@yuxihu yuxihu requested a review from szha as a code owner May 6, 2019 20:42
@yuxihu
Copy link
Member Author

yuxihu commented May 6, 2019

@szha @pengzhao-intel @TaoLv Please help review.

@TaoLv
Copy link
Member

TaoLv commented May 7, 2019

Thank you for the fix @yuxihu. Merging now.

@TaoLv TaoLv merged commit fdd45cf into apache:master May 7, 2019
@yuxihu yuxihu deleted the add_mkldnn_version branch May 7, 2019 16:02
access2rohit pushed a commit to access2rohit/incubator-mxnet that referenced this pull request May 14, 2019
* add mkldnn_version.h to pip package

* add headers from build/install/include folder
haohuanw pushed a commit to haohuanw/incubator-mxnet that referenced this pull request Jun 23, 2019
* add mkldnn_version.h to pip package

* add headers from build/install/include folder
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants