Skip to content

improve isdefined precision for 0 field types#58220

Merged
oscardssmith merged 10 commits intoJuliaLang:masterfrom
oscardssmith:os/improve-isdefined-inference
May 3, 2025
Merged

improve isdefined precision for 0 field types#58220
oscardssmith merged 10 commits intoJuliaLang:masterfrom
oscardssmith:os/improve-isdefined-inference

Conversation

@oscardssmith
Copy link
Copy Markdown
Member

alternate to #58214.

oscarddssmith added 2 commits April 24, 2025 17:20
@oscardssmith oscardssmith added performance Must go faster compiler:inference Type inference labels Apr 24, 2025
Co-authored-by: Jeff Bezanson <jeff.bezanson@gmail.com>
Co-authored-by: Jeff Bezanson <jeff.bezanson@gmail.com>
Copy link
Copy Markdown
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.

What about Tuple? Those can throw also

@vtjnash vtjnash added merge me PR is reviewed. Merge when all tests are passing backport 1.12 Change should be backported to release-1.12 labels May 2, 2025
@oscardssmith oscardssmith merged commit 1237a9c into JuliaLang:master May 3, 2025
4 of 7 checks passed
@oscardssmith oscardssmith deleted the os/improve-isdefined-inference branch May 3, 2025 00:46
aviatesk pushed a commit that referenced this pull request May 3, 2025
alternate to #58214.

---------

Co-authored-by: Jeff Bezanson <jeff.bezanson@gmail.com>
@aviatesk aviatesk removed merge me PR is reviewed. Merge when all tests are passing backport 1.12 Change should be backported to release-1.12 labels May 3, 2025
@aviatesk aviatesk mentioned this pull request May 3, 2025
53 tasks
charleskawczynski pushed a commit to charleskawczynski/julia that referenced this pull request May 12, 2025
alternate to JuliaLang#58214.

---------

Co-authored-by: Jeff Bezanson <jeff.bezanson@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

compiler:inference Type inference performance Must go faster

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants