Skip to content

Allow MKLDNN on macOS, and any other OS where CMake is able to detect it.#9638

Closed
resistor wants to merge 1 commit intopytorch:masterfrom
resistor:mkldnn-macos
Closed

Allow MKLDNN on macOS, and any other OS where CMake is able to detect it.#9638
resistor wants to merge 1 commit intopytorch:masterfrom
resistor:mkldnn-macos

Conversation

@resistor
Copy link
Contributor

No description provided.

@soumith
Copy link
Collaborator

soumith commented Jul 20, 2018

cc: @cpuhrsch do you remember or know why we disabled MKLDNN on non-Linux platforms?

@resistor
Copy link
Contributor Author

@soumith Looking through commit history, it was never explicitly disabled. The code was originally authored as Linux-only, and then Windows support was added.

@cpuhrsch
Copy link
Contributor

@soumith - No, that code was before my time.

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@soumith is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

jramseyer pushed a commit to jramseyer/pytorch that referenced this pull request Jul 30, 2018
… it.

Summary: Pull Request resolved: pytorch#9638

Reviewed By: soumith

Differential Revision: D8946130

Pulled By: resistor

fbshipit-source-id: 87bd9cb12608467b05bd4998fdb00bfdbd038ca2
goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
… it.

Summary: Pull Request resolved: pytorch#9638

Reviewed By: soumith

Differential Revision: D8946130

Pulled By: resistor

fbshipit-source-id: 87bd9cb12608467b05bd4998fdb00bfdbd038ca2
@ezyang ezyang added the merged label Jun 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants