Skip to content
This repository has been archived by the owner on Aug 11, 2020. It is now read-only.

[WIP] Change MKL BLAS linkage from LP64 to ILP64 #365

Closed
wants to merge 7 commits into from

Conversation

TaoLv
Copy link
Member

@TaoLv TaoLv commented Dec 14, 2018

To support large tensor.

Need #343 be merged at first. Another PR should be raised for MXNet accordingly.

Reference:
https://software.intel.com/en-us/mkl-macos-developer-guide-using-the-ilp64-interface-vs-lp64-interface

@TaoLv
Copy link
Member Author

TaoLv commented Dec 26, 2018

MXNet: apache/mxnet#13723

@pengzhao-intel
Copy link

@TaoLv could you rebase the code and run CI again?

@szha this issue is detected by several cases when we are using Intel MKL library.
Could you help take a look? Thanks

@szha
Copy link
Member

szha commented Mar 6, 2019

#343 is merged. Please rebase your change and open a PR on mxnet for verification.

@TaoLv TaoLv mentioned this pull request Mar 11, 2019
@TaoLv
Copy link
Member Author

TaoLv commented Mar 11, 2019

MXNet verification PR: apache/mxnet#13723
I'm also thinking to add a flag for user to choose LP64 or ILP64, instead of making ILP64 default.

@szha
Copy link
Member

szha commented Aug 4, 2019

This code base has been donated to the Apache MXNet project per #373, and repo is deprecated. Future development should continue in Apache MXNet.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@szha szha closed this Jul 26, 2020
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.

4 participants