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

Set intersection = 1 during intersect_sub_datatype #46882

Merged
merged 1 commit into from
Sep 24, 2022
Merged

Conversation

N5N3
Copy link
Member

@N5N3 N5N3 commented Sep 24, 2022

At present, the subtype_in_env check is performed without this flag, thus var_lt/var_gt might set the wrong bounds.
close #46871.

At present, the `subtype_in_env` check is performed without this flag, thus `var_lt`/`var_gt` might set the wrong bounds.
Copy link
Sponsor Member

@vtjnash vtjnash left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Amazing find

@Keno Keno added backport 1.8 Change should be backported to release-1.8 types and dispatch Types, subtyping and method dispatch labels Sep 24, 2022
@N5N3 N5N3 merged commit e6d9979 into JuliaLang:master Sep 24, 2022
@N5N3 N5N3 deleted the tempe branch September 24, 2022 14:32
@vtjnash
Copy link
Sponsor Member

vtjnash commented Sep 24, 2022

We should perhaps check if this closed any others too!

@thchr
Copy link
Contributor

thchr commented Sep 24, 2022

Would a backport to v1.6 be possible as well?

@N5N3
Copy link
Member Author

N5N3 commented Sep 24, 2022

Would a backport to v1.6 be possible as well?

Of course!

We should perhaps check if this closed any others too!

I took a quick look, but none of them looks related. Maybe we should add known intersect/subtype bug to the test file.

@N5N3 N5N3 added the backport 1.6 Change should be backported to release-1.6 label Sep 24, 2022
KristofferC pushed a commit that referenced this pull request Sep 30, 2022
@KristofferC KristofferC mentioned this pull request Sep 30, 2022
37 tasks
N5N3 added a commit to N5N3/julia that referenced this pull request Sep 30, 2022
KristofferC pushed a commit that referenced this pull request Oct 28, 2022
@KristofferC KristofferC removed the backport 1.8 Change should be backported to release-1.8 label Nov 8, 2022
KristofferC pushed a commit that referenced this pull request Dec 21, 2022
KristofferC pushed a commit that referenced this pull request Dec 21, 2022
KristofferC pushed a commit that referenced this pull request Dec 21, 2022
KristofferC pushed a commit that referenced this pull request Dec 21, 2022
staticfloat pushed a commit that referenced this pull request Dec 23, 2022
KristofferC pushed a commit that referenced this pull request Oct 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 1.6 Change should be backported to release-1.6 types and dispatch Types, subtyping and method dispatch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unexpected Illegal instruction error
5 participants