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

Added blas and int64 to cmake options #15739

Closed
wants to merge 2 commits into from
Closed

Added blas and int64 to cmake options #15739

wants to merge 2 commits into from

Conversation

ChaiBapchya
Copy link
Contributor

@ChaiBapchya ChaiBapchya commented Aug 2, 2019 •
edited
Loading

Description

For building MXNet, while using

./dev_menu.py build

It picks up config for cmake from cmake/cmake_options.yml
It doesn't have 2 build flags for BLAS and INT64_TENSOR

Checklist

Essentials

Please feel free to remove inapplicable items for your PR.

  • Changes are complete (i.e. I finished coding on this PR)
  • Code is well-documented:
  • To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

@ChaiBapchya ChaiBapchya requested a review from szha as a code owner August 2, 2019 23:17
@larroy larroy mentioned this pull request Aug 6, 2019
Closed
@ChaiBapchya
Copy link
Contributor Author

Closing since taken care of by Pedro's PR

@ChaiBapchya ChaiBapchya closed this Aug 6, 2019
@ChaiBapchya ChaiBapchya deleted the update_cmake_options branch August 6, 2019 22:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@mseth10 mseth10 mseth10 left review comments

@szha szha Awaiting requested review from szha szha is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ChaiBapchya @mseth10