Skip to content

test: disallow llvm-bcanalyzer and use new macro#88399

Merged
compnerd merged 1 commit intoswiftlang:mainfrom
compnerd:analyzer
Apr 14, 2026
Merged

test: disallow llvm-bcanalyzer and use new macro#88399
compnerd merged 1 commit intoswiftlang:mainfrom
compnerd:analyzer

Conversation

@compnerd
Copy link
Copy Markdown
Member

@compnerd compnerd commented Apr 9, 2026

This ensures that we use the correct llvm-bcanalyzer from the just built compiler tools.

@compnerd compnerd requested a review from a team as a code owner April 9, 2026 19:51
@compnerd
Copy link
Copy Markdown
Member Author

compnerd commented Apr 9, 2026

@swift-ci please smoke test

@compnerd
Copy link
Copy Markdown
Member Author

compnerd commented Apr 9, 2026

@swift-ci please smoke test macOS platform

@compnerd compnerd requested a review from etcwilde April 10, 2026 00:25
@etcwilde
Copy link
Copy Markdown
Member

Missed a test using the llvm-bcanalyzer:

[2026-04-10T01:56:51.991Z] # .---command stdout------------
[2026-04-10T01:56:51.991Z] # | *** Do not use llvm-bcanalyzer in tests; use %llvm-bcanalyzer. ***
[2026-04-10T01:56:51.991Z] # `-----------------------------
[2026-04-10T01:56:51.991Z] # executed command: exit 1
[2026-04-10T01:56:51.991Z] # .---command stderr------------
[2026-04-10T01:56:51.991Z] # | 'exit': command not found
[2026-04-10T01:56:51.991Z] # `-----------------------------
[2026-04-10T01:56:51.991Z] # error: command failed with exit status: 127

Will need to update swift/validation-test/Serialization/bridging-header-first.swift.

This ensures that we use the correct llvm-bcanalyzer from the just built
compiler tools.
@compnerd
Copy link
Copy Markdown
Member Author

@swift-ci please smoke test

@compnerd compnerd merged commit 7cc70e6 into swiftlang:main Apr 14, 2026
3 checks passed
@compnerd compnerd deleted the analyzer branch April 14, 2026 15:45
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.

3 participants