Skip to content

[Sanitizer] Add new flag -sanitize-stable-abi flag to link against the #1509

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
Dec 18, 2023

Conversation

wrotki
Copy link
Contributor

@wrotki wrotki commented Dec 15, 2023

new sanitiers stable ABI in compiler-rt.

Compiler-rt recently introduced a stable ABI for sanitizers. This patch adds a flag to pass this flag to swift-frontend during compilation and clang at link time to link against the new stable ABI.

NOTE: this was #1495 , but Usama is out till after the new year break, so I need to take over and apply feedback from there

rdar://118997873

@wrotki wrotki requested a review from artemcm December 15, 2023 23:16
@wrotki
Copy link
Contributor Author

wrotki commented Dec 15, 2023

@swift-ci test

@wrotki
Copy link
Contributor Author

wrotki commented Dec 16, 2023

@swift-ci test

new sanitiers stable ABI in compiler-rt.

Compiler-rt recently introduced a stable ABI for sanitizers. This patch
adds a flag to pass this flag to swift-frontend during compilation and
clang at link time to link against the new stable ABI.

rdar://118997873
@wrotki wrotki force-pushed the eng/m_borsa/118997873 branch from 86f23ba to bc20486 Compare December 16, 2023 02:27
@wrotki
Copy link
Contributor Author

wrotki commented Dec 16, 2023

@swift-ci test

@wrotki
Copy link
Contributor Author

wrotki commented Dec 16, 2023

@swift-ci Please test Windows platform

@wrotki wrotki merged commit 2e355ae into main Dec 18, 2023
@wrotki wrotki deleted the eng/m_borsa/118997873 branch December 18, 2023 17:59
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