Skip to content

Optimize evaluate - #857

Open
dsharlet wants to merge 2 commits into
mainfrom
ds/parallel-1
Open

Optimize evaluate#857
dsharlet wants to merge 2 commits into
mainfrom
ds/parallel-1

Conversation

@dsharlet

@dsharlet dsharlet commented Aug 1, 2026

Copy link
Copy Markdown
Owner
  • Remove evaluator class and just pass eval_context to each eval function.
  • Skip evaluating the closure when n = 1 in a parallel loop.
  • Avoid reference counting stmt objects when not necessary

dsharlet added 2 commits August 1, 2026 13:45
@dsharlet
Remove evaluator class to just pass eval_context
dfb0f3d 
This is a small speedup of some of the `eval` functions, because the eval context is now stored in a register.
@dsharlet
Skip evaluating the closure in the n = 1 case of a parallel loop
45401ac
@dsharlet
dsharlet requested a review from vksnk August 1, 2026 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vksnk vksnk Awaiting requested review from vksnk

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@dsharlet