google / google/xls

Reduce AST presence in Bytecode[Interpreter]

Open
#608 0 comments 0 reactions 1 assignee Claimed by @dplassgit View on GitHub
cleanup dslx
Dominant language
C++
Stars
1.9k
Forks
283
Avg merge
2d 10h
Merged PRs (30d)
135

Description

e.g., in SpawnData, we currently have (or will shortly have) Spawn* and Proc* elements. The Spawn* is used for TypeInfo lookup inside the interpreter and the Proc* is used for extracting the `config` and `next` functions. If instead we just passed the TypeInfos and the `config` and `next` functions, we could reduce the AST surface exposed inside BytecodeInterpreter.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.