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

Disable BasicFunctionUsedUnusedTwoRuntimes in Apple ASAN #1351

Closed
wants to merge 1 commit into from

Conversation

avp
Copy link
Contributor

@avp avp commented Mar 14, 2024

Summary:
ASAN has a big minimum native stack gap, which causes this test to crash
on mac under ASAN due to the use of std::async.
Disable this test under ASAN on Mac.

Differential Revision: D54917788

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Mar 14, 2024
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D54917788

Summary:

ASAN has a big minimum native stack gap, which causes this test to crash
on mac under ASAN due to the use of `std::async.`
Disable this test under ASAN on Mac.

Differential Revision: D54917788
@avp avp force-pushed the export-D54917788-to-static_h branch from 79d2227 to 7be237d Compare March 14, 2024 21:27
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D54917788

facebook-github-bot pushed a commit that referenced this pull request Mar 14, 2024
Summary:
Pull Request resolved: #1351

ASAN has a big minimum native stack gap, which causes this test to crash
on mac under ASAN due to the use of `std::async.`
Disable this test under ASAN on Mac.

Reviewed By: neildhar

Differential Revision: D54917788

fbshipit-source-id: f87b4855daeda89ce7f1437d874326eed36a4b50
@avp avp closed this Aug 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Do not delete this pull request or issue due to inactivity. fb-exported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants