Skip to content

Route ordinary calls through unified expression loop - #2984

Merged
rogeralsing merged 1 commit into
mainfrom
codex/unified-bytecode-loop-27
Jun 1, 2026
Merged

Route ordinary calls through unified expression loop#2984
rogeralsing merged 1 commit into
mainfrom
codex/unified-bytecode-loop-27

Conversation

@rogeralsing

Copy link
Copy Markdown
Contributor

Summary

  • lower ordinary non-optional identifier/member/super call-target opcodes through the unified general expression loop
  • keep direct eval and optional-call shapes on the first-boundary helper because they carry directness metadata or packed nullish jump targets
  • shrink the checked general expression lowering gap list to JumpIfShortCircuited

Verification

  • rtk dotnet test tests/Asynkron.JsEngine.Tests -c Release --filter 'FullyQualifiedNameExpressionProgramCoverageMapTests|FullyQualifiedNameUnifiedBytecodeProductionEligibilityTests|FullyQualifiedNameUnifiedBytecodeProductionInvocationTests|FullyQualifiedNameUnifiedBytecodePrototypeTests' (739 passed)
  • rtk rg 'EvaluateExpression(|ProfileEvaluateExpression(' src/Asynkron.JsEngine/Ast/TypedAstEvaluator.ExecutionPlanRunner* (no matches)
  • rtk ./tools/profile forloop --memory (Total allocated 6.86 MB)
  • rtk git diff --check

Copilot AI review requested due to automatic review settings June 1, 2026 14:32
@rogeralsing
rogeralsing merged commit b460b1b into main Jun 1, 2026
@rogeralsing
rogeralsing deleted the codex/unified-bytecode-loop-27 branch June 1, 2026 14:32

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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.

2 participants