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

Fix stability inference flags for non-jvm #1006

Open
wants to merge 18 commits into
base: release/compiler-integration
Choose a base branch
from

Conversation

eymar
Copy link
Member

@eymar eymar commented Jan 19, 2024

No description provided.

igordmn and others added 18 commits January 18, 2024 10:59
(not original commit. it's squashed with other changes during rebases)
Change-Id: I983a376b3df1de74346568d9eae805aff67a1557
Disable for overridden symbols too.

Disable decoys generation for SAM in fun interfaces

fun interfaces can contain only 1 SAM, but decoys generation would add another one (so 2 in total), which leads to an error in further kotlin lowerings (SAM transformation).

With this commit, fun interface(s) won't be affected by decoys generation and decoys substitution.
Add HiddenFromObjC to property getter and setter if necessary (#579)

Use FqName in HideFromObjCDeclarationsSet

Also: mark properties of CompositionLocal type as HiddenFromObjC
This commit removes extra `copyWithNewTypeParams` for calls. It led to new type generation for an anonymous object which didn't the type of 'val a'.

Note: Run all tests + tried some new cases, and tested with compose:mpp:demo. This change didn't reveal any new issues. But there is chance it can break something we don't know yet.
@eymar eymar requested a review from shishkin-pavel January 19, 2024 13:48
@eymar
Copy link
Member Author

eymar commented Jan 19, 2024

Failing CI is okay in this branch - this branch contains only compiler plugin related changes.

@eymar eymar force-pushed the release/compiler-integration branch from 05316ee to bccd756 Compare February 14, 2024 15:17
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