Skip to content

[NFC] Update span unit tests for bounds check optimizations #79222

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 3 commits into from
Feb 10, 2025

Conversation

meg-gupta
Copy link
Contributor

@meg-gupta meg-gupta commented Feb 7, 2025

Add -O to the compilation of SpanExtras and update check lines especially check lines for llvm's vectorization with the updated version.

Using -O during module compilation is important because of how preconditions are lowered in -Onone vs -O. Even though we run optimizations on an imported module, it is important to use -O for module compilation especially when we are analyzing optimization variations.

@meg-gupta
Copy link
Contributor Author

@swift-ci smoke test

These initializers have ownership errors. Once rdar://144352938 is fixed,
these can be enabled again.
@meg-gupta meg-gupta force-pushed the boundstestsupdate branch 2 times, most recently from add1ba3 to 0bff081 Compare February 9, 2025 18:53
@meg-gupta
Copy link
Contributor Author

@swift-ci test

@meg-gupta
Copy link
Contributor Author

@swift-ci test

@meg-gupta meg-gupta marked this pull request as ready for review February 10, 2025 15:04
@meg-gupta meg-gupta requested a review from eeckstein as a code owner February 10, 2025 15:04
@meg-gupta meg-gupta merged commit 35f10fa into swiftlang:main Feb 10, 2025
5 checks passed
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.

1 participant