Skip to content

set CMAKE_HIP_ARCHIVE_APPEND#9394

Closed
bddppq wants to merge 1 commit intopytorch:masterfrom
bddppq:caffe2-cmake-hip-fix
Closed

set CMAKE_HIP_ARCHIVE_APPEND#9394
bddppq wants to merge 1 commit intopytorch:masterfrom
bddppq:caffe2-cmake-hip-fix

Conversation

@bddppq
Copy link
Contributor

@bddppq bddppq commented Jul 12, 2018

@petrex

To make -DBUILD_SHARED_LIBS=OFF working

@bddppq bddppq requested a review from Jorghi12 July 12, 2018 18:07
@bddppq
Copy link
Contributor Author

bddppq commented Jul 12, 2018

@Jorghi12 I remember you have an pull request to upstream all these cmake variables to the rocm repo? Could you add this line to it? (if it's already merged I can create another upstream PR)

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.

@bddppq has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

zdevito pushed a commit to zdevito/ATen that referenced this pull request Jul 12, 2018
Summary:
petrex

To make `-DBUILD_SHARED_LIBS=OFF` working
Pull Request resolved: pytorch/pytorch#9394

Reviewed By: mingzhe09088

Differential Revision: D8822947

Pulled By: bddppq

fbshipit-source-id: 4fb213c723138804fb0fdb3b381e32623cf14468
zdevito pushed a commit to zdevito/ATen that referenced this pull request Jul 13, 2018
Summary:
petrex

To make `-DBUILD_SHARED_LIBS=OFF` working
Pull Request resolved: pytorch/pytorch#9394

Reviewed By: mingzhe09088

Differential Revision: D8822947

Pulled By: bddppq

fbshipit-source-id: 4fb213c723138804fb0fdb3b381e32623cf14468
@Jorghi12
Copy link
Contributor

@bddppq Here's the upstreamed merge: https://github.com/ROCm-Developer-Tools/HIP/pull/558/files

goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
Summary:
petrex

To make `-DBUILD_SHARED_LIBS=OFF` working
Pull Request resolved: pytorch#9394

Reviewed By: mingzhe09088

Differential Revision: D8822947

Pulled By: bddppq

fbshipit-source-id: 4fb213c723138804fb0fdb3b381e32623cf14468
@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.

4 participants