Skip to content

Conversation

@aviatesk
Copy link
Member

@aviatesk aviatesk commented Apr 7, 2025

By moving the optional assume_bindings_static refinement logic into abstract_eval_partition_load and removing the redirect via abstract_eval_globalref_partition.
Also adds test cases with assume_bindings_static=true.

@aviatesk aviatesk requested a review from Keno April 7, 2025 00:01
@aviatesk aviatesk added the backport 1.12 Change should be backported to release-1.12 label Apr 7, 2025
By moving the optional `assume_bindings_static` refinement logic into
`abstract_eval_globalref_partition` and removing the redirect via
`abstract_eval_globalref_partition`.
Also adds test cases with `assume_bindings_static=true`.
@aviatesk aviatesk force-pushed the avi/simplify-abstract_eval_globalref branch from e06ad83 to b2947aa Compare April 8, 2025 18:22
@Keno
Copy link
Member

Keno commented Apr 8, 2025

Change is fine by me. I don't know that I can say it is simpler, but if it helps for JET, I'm happy with the change.

@KristofferC KristofferC mentioned this pull request Apr 9, 2025
51 tasks
@aviatesk aviatesk merged commit 4480f42 into master Apr 9, 2025
7 checks passed
@aviatesk aviatesk deleted the avi/simplify-abstract_eval_globalref branch April 9, 2025 12:36
aviatesk added a commit that referenced this pull request Apr 9, 2025
By moving the optional `assume_bindings_static` refinement logic into
`abstract_eval_partition_load` and removing the redirect via
`abstract_eval_globalref_partition`.
Also adds test cases with `assume_bindings_static=true`.
@KristofferC KristofferC removed the backport 1.12 Change should be backported to release-1.12 label Apr 25, 2025
serenity4 pushed a commit to serenity4/julia that referenced this pull request May 1, 2025
By moving the optional `assume_bindings_static` refinement logic into
`abstract_eval_partition_load` and removing the redirect via
`abstract_eval_globalref_partition`.
Also adds test cases with `assume_bindings_static=true`.
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.

4 participants