Skip to content

Test: Expand -target-min-inlining-versiong tests to cover more corner cases #58679

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

Conversation

tshortli
Copy link
Contributor

@tshortli tshortli commented May 5, 2022

Expand -target-min-inlining-versiong tests to cover a number of corner cases:

  • unavailable declarations and unavailable containers
  • SPI declarations and spi containers
  • property initializer expressions
  • property wrappers

The tests document existing behavior of the availability checker, including several bugs. The FIXMEs in the test serve as a todo list of issues to resolve before the flag can be adopted widely.

…f corner cases:

- unavailable declarations and unavailable containers
- SPI declarations and spi containers
- property initializer expressions
- property wrappers

The tests document existing behavior of the availability checker, including several bugs. The FIXMEs in the test serve as a todo list of issues to resolve before the flag can be adopted widely.
@tshortli
Copy link
Contributor Author

tshortli commented May 5, 2022

@swift-ci please test

@tshortli tshortli merged commit b9be8b0 into swiftlang:main May 5, 2022
@tshortli tshortli deleted the target-min-inlinig-version-expanded-testcases branch May 5, 2022 20:33
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