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.

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

Basic Compiler Optimizations 3: Revenge of the LINQ #1765

Merged
merged 33 commits into from
Jun 15, 2024

Conversation

ike709
Copy link
Collaborator

@ike709 ike709 commented May 1, 2024

Revives #1630 on latest OD (and rips out the bytecode printer).

A few opcodes have been removed and Rgb has been added since the initial PR, and the code has been updated accordingly.

@boring-cyborg boring-cyborg bot added Compiler Involves the OpenDream compiler Runtime Involves the OpenDream server/runtime labels May 1, 2024
@ike709 ike709 requested a review from wixoaGit May 1, 2024 01:58
@ike709
Copy link
Collaborator Author

ike709 commented May 1, 2024

Ripped out the remaining optimizer LINQ and shaved another 2 seconds off compiletime.

@ike709 ike709 changed the title Basic Compiler Optimizations 3: Revenge of the Parameters Basic Compiler Optimizations 3: Revenge of the LINQ May 1, 2024
Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

InspectCode found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

@wixoaGit wixoaGit merged commit d127eff into OpenDreamProject:master Jun 15, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Compiler Involves the OpenDream compiler Runtime Involves the OpenDream server/runtime size/XXL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants