Skip to content

Add install for libCaffe2_perfkernels_avx*.a#53825

Closed
gemfield wants to merge 1 commit intopytorch:masterfrom
CivilNet:fix_static_lib_install
Closed

Add install for libCaffe2_perfkernels_avx*.a#53825
gemfield wants to merge 1 commit intopytorch:masterfrom
CivilNet:fix_static_lib_install

Conversation

@gemfield
Copy link
Copy Markdown
Contributor

@gemfield gemfield commented Mar 11, 2021

When build libtorch static library, these three static libraries will be generated but won't be installed to CMAKE_INSTALL_LIBDIR:

  • libCaffe2_perfkernels_avx2.a
  • libCaffe2_perfkernels_avx512.a
  • libCaffe2_perfkernels_avx.a

This PR will fix this issue.

Please be noted that after this fix there still have static libraries missing in CMAKE_INSTALL_LIBDIR, but they belong to third_party repo, and we need to fix in the corresponding repo:

  • libfoxi_loader.a
  • libonnx.a
  • libonnx_proto.a
  • libfmt.a
  • libnccl_static.a

@facebook-github-bot
Copy link
Copy Markdown
Contributor

facebook-github-bot commented Mar 11, 2021

💊 CI failures summary and remediations

As of commit 240130c (more details on the Dr. CI page):


  • 1/2 failures possibly* introduced in this PR
    • 1/1 non-scanned failure(s)
  • 1/2 broken upstream at merge base 0a549f9 from Mar 10 until Mar 11

🚧 1 fixed upstream failure:

These were probably caused by upstream breakages that were already fixed.

Please rebase on the viable/strict branch (expand for instructions)

If your commit is older than viable/strict, run these commands:

git fetch https://github.com/pytorch/pytorch viable/strict
git rebase FETCH_HEAD

ci.pytorch.org: 1 failed


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions to the (internal) Dr. CI Users group.

@gemfield
Copy link
Copy Markdown
Contributor Author

@malfet could you have a code review?

Copy link
Copy Markdown
Contributor

@malfet malfet left a comment

Choose a reason for hiding this comment

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

Looks good to me

Copy link
Copy Markdown
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.

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

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@malfet merged this pull request in 700c817.

@gemfield gemfield deleted the fix_static_lib_install branch March 18, 2021 16:24
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
When build libtorch static library, these three static libraries will be generated but won't be installed to CMAKE_INSTALL_LIBDIR:
- libCaffe2_perfkernels_avx2.a
- libCaffe2_perfkernels_avx512.a
- libCaffe2_perfkernels_avx.a

This PR will fix this issue.

Please be noted that after this fix there still have static libraries missing in CMAKE_INSTALL_LIBDIR, but they belong to third_party repo, and we need to fix in the corresponding repo:
- libfoxi_loader.a
- libonnx.a
- libonnx_proto.a
- libfmt.a
- libnccl_static.a

Pull Request resolved: pytorch#53825

Reviewed By: ngimel

Differential Revision: D27013844

Pulled By: malfet

fbshipit-source-id: 8a84cc72b6ae87393ca26c4e474f5526a7b18ab2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants