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

Inline recursive calls #1022

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

Inline recursive calls #1022

wants to merge 7 commits into from

Conversation

JonasWanke
Copy link
Member

@JonasWanke JonasWanke commented Apr 18, 2024

Checklist

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

@JonasWanke JonasWanke added the T: Performance Type: Performance Improvements label Apr 18, 2024
@jwbot jwbot enabled auto-merge April 18, 2024 12:43
@jwbot jwbot added P: Core Package: Candy's standard library P: Compiler: Frontend Package: The compiler frontend P: FileSystem Package: File System Interaction labels Apr 18, 2024
@JonasWanke JonasWanke marked this pull request as draft April 18, 2024 12:43
auto-merge was automatically disabled April 18, 2024 12:43

Pull request was converted to draft

@jwbot
Copy link
Collaborator

jwbot commented Apr 18, 2024

The golden IRs have changed: 8c4d446..52b6f95

@jwbot
Copy link
Collaborator

jwbot commented Apr 18, 2024

🐰Bencher

ReportThu, April 18, 2024 at 14:25:54 UTC
ProjectCandy
Branchinline-recursive-calls
TestbedGitHub Actions: Ubuntu 22.04
BenchmarkEstimated CyclesEstimated Cycles Results
estimated cycles
InstructionsInstructions Results
instructions
L1 AccessesL1 Accesses Results
accesses
L2 AccessesL2 Accesses Results
accesses
RAM AccessesRAM Accesses Results
accesses
Total AccessesTotal Accesses Results
total-accesses
compile: Examples/fibonacci.candy ➖ (view plot)1879403693.000➖ (view plot)1247172137.000➖ (view plot)1694371748.000➖ (view plot)11055632.000➖ (view plot)3707251.000➖ (view plot)1709134631.000
compile: Examples/helloWorld.candy ➖ (view plot)286110638.000➖ (view plot)177227743.000➖ (view plot)249474893.000➖ (view plot)1479650.000➖ (view plot)835357.000➖ (view plot)251789900.000
vm_runtime: Examples/fibonacci.candy 10➖ (view plot)185606034.000➖ (view plot)108527870.000➖ (view plot)152857549.000➖ (view plot)817250.000➖ (view plot)818921.000➖ (view plot)154493720.000
vm_runtime: Examples/helloWorld.candy ➖ (view plot)157066809.000➖ (view plot)90108091.000➖ (view plot)127036044.000➖ (view plot)702169.000➖ (view plot)757712.000➖ (view plot)128495925.000

Bencher - Continuous Benchmarking
View Public Perf Page
Docs | Repo | Chat | Help

@jwbot
Copy link
Collaborator

jwbot commented Apr 18, 2024

The golden IRs have changed: 8c4d446..9270689

@jwbot
Copy link
Collaborator

jwbot commented Apr 18, 2024

The golden IRs have changed: 8c4d446..15c6393

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P: Compiler: Frontend Package: The compiler frontend P: Core Package: Candy's standard library P: FileSystem Package: File System Interaction T: Performance Type: Performance Improvements
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants