Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

look into memoizing some @typeInfo results #17689

Open
andrewrk opened this issue Oct 24, 2023 · 0 comments
Open

look into memoizing some @typeInfo results #17689

andrewrk opened this issue Oct 24, 2023 · 0 comments
Labels
enhancement Solving this issue will likely involve adding new logic or components to the codebase. frontend Tokenization, parsing, AstGen, Sema, and Liveness. optimization
Milestone
0.14.0

Comments

@andrewrk
Copy link
Member

Extracting some TODO comments into an issue (#363):

$ grep -RI 'TODO: look into' ../src/
../src/Sema.zig:            // TODO: look into memoizing this result.
../src/Sema.zig:            // TODO: look into memoizing this result.
../src/Sema.zig:            // TODO: look into memoizing this result.
../src/Sema.zig:            // TODO: look into memoizing this result.
../src/Sema.zig:            // TODO: look into memoizing this result.

These are all in the implementation of @typeInfo.
@andrewrk andrewrk added enhancement Solving this issue will likely involve adding new logic or components to the codebase. optimization frontend Tokenization, parsing, AstGen, Sema, and Liveness. labels Oct 24, 2023
@andrewrk andrewrk added this to the 0.13.0 milestone Oct 24, 2023
andrewrk added a commit that referenced this issue Oct 24, 2023
@andrewrk
Sema: move some TODO comments to an issue
6df4bfd 
Tracking issue: #17689
andrewrk added a commit that referenced this issue Oct 25, 2023
@andrewrk
Sema: move some TODO comments to an issue
9e49477 
Tracking issue: #17689
andrewrk added a commit that referenced this issue Oct 26, 2023
@andrewrk
Sema: move some TODO comments to an issue
426b598 
Tracking issue: #17689
@mlugg mlugg moved this to Uncategorized in Performance Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement Solving this issue will likely involve adding new logic or components to the codebase. frontend Tokenization, parsing, AstGen, Sema, and Liveness. optimization
Projects
Performance
Status: Uncategorized
Milestone
0.14.0
Development

No branches or pull requests
1 participant
@andrewrk