Skip to content
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

[CMake] allow custom options when building the compiler with debug info #73603

Merged
merged 1 commit into from
May 15, 2024

Conversation

edymtt
Copy link
Contributor

@edymtt edymtt commented May 13, 2024

Currently those are hardcoded to -g, but in some Apple internal configurations we would like to change them.

There are other part of the build system that hardcode -g (e.g. in SwiftCompilerSources and AddSwiftStdlib.cmake), but we are not interested in those at the moment -- we will address those in the future if need be.

Supports rdar://127503136

@edymtt
Copy link
Contributor Author

edymtt commented May 13, 2024

@swift-ci please test

@edymtt
Copy link
Contributor Author

edymtt commented May 13, 2024

@swift-ci please test WebAssembly

@edymtt
Copy link
Contributor Author

edymtt commented May 13, 2024

@swift-ci please build toolchain

@edymtt
Copy link
Contributor Author

edymtt commented May 14, 2024

The WebAssembly failure is the same as the main job

/home/build-user/swift-nightly-install/usr/bin/swiftc: symbol lookup error: /home/build-user/build/buildbot_linux/earlyswiftdriver-linux-x86_64/release/dependencies/swift-tools-support-core/lib/libTSCBasic.so: undefined symbol: $s8Dispatch0A5QueueC5async5group3qos5flags7executeyAA0A5GroupCSg_AA0A3QoSVAA0A13WorkItemFlagsVyyXBtF

Currently those are hardcoded to `-g`, but in some Apple internal
configurations we would like to change them.

There are other part of the build system that hardcode `-g`
(e.g. in `SwiftCompilerSources` and `AddSwiftStdlib.cmake`),
but we are not interested in those at the moment -- we will address those
in the future if need be.

Supports rdar://127503136
@edymtt edymtt force-pushed the edymtt/swift-args-for-debug-info-non-lto branch from 11b7dcc to ce0e620 Compare May 14, 2024 15:43
@edymtt
Copy link
Contributor Author

edymtt commented May 14, 2024

@swift-ci please test

@edymtt
Copy link
Contributor Author

edymtt commented May 14, 2024

@swift-ci please test WebAssembly

@edymtt
Copy link
Contributor Author

edymtt commented May 14, 2024

@swift-ci please build toolchain

@edymtt edymtt merged commit 5977d31 into apple:main May 15, 2024
8 of 9 checks passed
@edymtt edymtt deleted the edymtt/swift-args-for-debug-info-non-lto branch May 15, 2024 15:25
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.

None yet

2 participants