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

Backport "Fix isomorphism tests of AndOrTypes under non-empty BinderPairs" to LTS #21986

Merged
merged 1 commit into from
Dec 2, 2024

Conversation

prolativ
Copy link
Contributor

Backports #21017 to the LTS branch.

PR submitted by the release tooling.
[skip ci]

@prolativ prolativ force-pushed the backport-lts-3.3-18503 branch from 5f08ec6 to b1a81bb Compare November 21, 2024 11:54
…21017)

Before the changes, when comparing two `HKTypeLambda` result types, the
list of binding pairs was lost when entering comparison of `AndOrType`s,
which caused the `equals` to fail, and hence prevented hash-consing.

Even though `M1` and `M2` in pos/i20858-min should still conform to
one-another, we entered a deep-subtype comparison because of the order
in which the TypeComparer does dealiasing of AppliedTypes, and
comparison of MatchCases and AndTypes.

Fix #20858
[Cherry-picked f2829c3]
@prolativ prolativ force-pushed the backport-lts-3.3-21017 branch from 27fe00e to 1aca443 Compare November 21, 2024 11:56
Base automatically changed from backport-lts-3.3-18503 to lts-3.3 December 2, 2024 17:37
@WojciechMazur
Copy link
Contributor

No regressions detected in the community build up to backport-lts-3.3-21186.

Reference

@WojciechMazur WojciechMazur merged commit e50f921 into lts-3.3 Dec 2, 2024
19 checks passed
@WojciechMazur WojciechMazur deleted the backport-lts-3.3-21017 branch December 2, 2024 17:37
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