Skip to content
This repository was archived by the owner on Aug 15, 2025. It is now read-only.

Conversation

tinglvv
Copy link
Contributor

@tinglvv tinglvv commented May 21, 2024

original PR #1804

set CUDA_ARCH_LIST to 9.0 (hopper only) for cuda-aarch64.
fixed issues in copy path when creating the wheel.
set MAX_JOBS=5 to resolve OOM in building flash-attn.
remove libarm_compute_core.so from copy path per ARM-software/ComputeLibrary#1054 (comment).
move copy path for libopenblas.so from conda to /opt/OpenBLAS/lib/ due to #1803.
enabled linker script optimization.

Copy link
Contributor

@nWEIdia nWEIdia left a comment

Choose a reason for hiding this comment

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

Looks great!

Copy link
Contributor

@atalman atalman left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you @tinglvv

@atalman atalman merged commit 465d98b into pytorch:main May 22, 2024
pytorchmergebot pushed a commit to pytorch/pytorch that referenced this pull request May 27, 2024
rebasing #124112.
too many conflict files, so starting a new PR.

Test pytorch/builder#1775 (merged) for ARM wheel addition
Test pytorch/builder#1828 (merged) for setting MAX_JOBS

Current issue to follow up:
#126980

Co-authored-by: Aidyn-A <aidyn.b.aitzhan@gmail.com>
Pull Request resolved: #126174
Approved by: https://github.com/nWEIdia, https://github.com/atalman
titaiwangms pushed a commit to titaiwangms/pytorch that referenced this pull request May 28, 2024
rebasing pytorch#124112.
too many conflict files, so starting a new PR.

Test pytorch/builder#1775 (merged) for ARM wheel addition
Test pytorch/builder#1828 (merged) for setting MAX_JOBS

Current issue to follow up:
pytorch#126980

Co-authored-by: Aidyn-A <aidyn.b.aitzhan@gmail.com>
Pull Request resolved: pytorch#126174
Approved by: https://github.com/nWEIdia, https://github.com/atalman
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants