Skip to content

Conversation

@MaxDesiatov
Copy link
Contributor

This LLVM tool is required for cross-compiling and debugging binaries cross-compiled to non-Darwin platforms, but for some reason is currently not included unlike llvm-ar or llvm-ranlib.

This LLVM tool is required for cross-compiling and debugging binaries cross-compiled to non-Darwin platforms, but for some reason is currently not included unlike `llvm-ar` or `llvm-ranlib`.
@MaxDesiatov MaxDesiatov requested a review from bnbarham July 10, 2025 11:51
@MaxDesiatov MaxDesiatov added build-script Area → utils: The build script cross-compilation Area → utils: Cross-compilation of project sources WebAssembly Platform: WebAssembly labels Jul 10, 2025
@MaxDesiatov
Copy link
Contributor Author

@swift-ci smoke test

@MaxDesiatov
Copy link
Contributor Author

@swift-ci build toolchain

Copy link
Member

@compnerd compnerd left a comment

Choose a reason for hiding this comment

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

Yeap, Windows already ships this as well.

@MaxDesiatov
Copy link
Contributor Author

@swift-ci build toolchain macos

@MaxDesiatov
Copy link
Contributor Author

@swift-ci build toolchain windows

@MaxDesiatov MaxDesiatov merged commit 2103b4b into swiftlang:main Jul 11, 2025
6 checks passed
@MaxDesiatov MaxDesiatov deleted the maxd/include-llvm-nm branch July 11, 2025 08:12
MaxDesiatov added a commit to MaxDesiatov/swift that referenced this pull request Jul 22, 2025
Follow up to swiftlang#82945 that makes `llvm-nm` behavior consistent on all platforms.
MaxDesiatov added a commit that referenced this pull request Jul 22, 2025
…83228)

Follow up to #82945 that makes `llvm-nm` installation consistent on all platforms.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build-script Area → utils: The build script cross-compilation Area → utils: Cross-compilation of project sources WebAssembly Platform: WebAssembly

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants