Skip to content

[build] Don't execute dsymutil in parallel #33654

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 27, 2020

Conversation

edymtt
Copy link
Contributor

@edymtt edymtt commented Aug 26, 2020

dsymutil is already multithreaded and can be memory intensive --
execute it one at a time.

Addresses rdar://63892559

`dsymutil` is already multithreaded and can be memory intensive --
execute it one at a time.

Addresses rdar://63892559
@edymtt
Copy link
Contributor Author

edymtt commented Aug 26, 2020

@swift-ci Please Build Toolchain macOS Platform

@edymtt
Copy link
Contributor Author

edymtt commented Aug 26, 2020

@swift-ci Please Build Toolchain Linux Platform

@edymtt
Copy link
Contributor Author

edymtt commented Aug 26, 2020

@swift-ci please smoke test

@edymtt
Copy link
Contributor Author

edymtt commented Aug 27, 2020

@swift-ci please smoke test Linux

@edymtt
Copy link
Contributor Author

edymtt commented Aug 27, 2020

@swift-ci Please Build Toolchain macOS Platform

@edymtt
Copy link
Contributor Author

edymtt commented Aug 27, 2020

@swift-ci Please Build Toolchain Linux Platform

@swift-ci
Copy link
Contributor

Linux Toolchain (Ubuntu 16.04)
Download Toolchain
Git Sha - d0f63d4

Install command
tar zxf swift-PR-33654-438-ubuntu16.04.tar.gz
More info

@swift-ci
Copy link
Contributor

macOS Toolchain
Download Toolchain
Git Sha - d0f63d4

Install command
tar -zxf swift-PR-33654-650-osx.tar.gz --directory ~/

@edymtt edymtt merged commit 102d638 into swiftlang:master Aug 27, 2020
edymtt added a commit to edymtt/swift that referenced this pull request Oct 1, 2020
The change in swiftlang#33654 had the effect of not leveraging dsymutil
parallelism -- pass instead multiple file at once.

Addresses rdar://69550787
edymtt added a commit that referenced this pull request Oct 2, 2020
The change in #33654 had the effect of not leveraging dsymutil
parallelism -- pass instead multiple file at once.

Addresses rdar://69550787
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