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

Added more benchmarks #323

Merged
merged 3 commits into from Apr 16, 2020
Merged

Added more benchmarks #323

merged 3 commits into from Apr 16, 2020

Conversation

Razican
Copy link
Member

@Razican Razican commented Apr 16, 2020

We now have more benchmarks, and parsing benchmarks also take into
account lexing, since in the future, both will be done together.

I also changed a bit the structure of the benchmark files, so now it should be easier to understand. We have 3 files: lexer.rs, parser.rs and exec.rs, and each has its benchmarks.

I also upgraded and updated dependencies, so this should also fix #322.

We now have more benchmarks, and parsing benchmarks also take into
account lexing, since in the future, both will be done together.
@Razican Razican added the enhancement New feature or request label Apr 16, 2020
@github-actions
Copy link

Benchmark for e7e3f15

Click to view benchmark
Test PR Benchmark Master Benchmark %
Create Realm 368.3±22.82µs 381.0±56.58µs 97%
Expression (Lexer) 1730.9±126.64ns undefined 100%
Expression (Parser) 4.1±0.33µs 2.8±0.22µs 145%
Fibonacci (Execution) 4.3±0.21ms undefined 100%
For loop (Execution) 458.1±34.24µs undefined 100%
For loop (Lexer) 4.5±0.34µs undefined 100%
For loop (Parser) 12.7±1.04µs undefined 100%
Hello World (Execution) 387.3±30.77µs undefined 100%
Hello World (Lexer) 818.7±65.46ns 790.4±56.54ns 104%
Hello World (Parser) 2.0±0.20µs 1054.1±89.26ns 193%
Symbol Creation 475.7±42.73µs undefined 100%
Symbol creation (Execution) 465.2±46.50µs undefined 100%
fibonacci (Execution) 4.2±0.23ms undefined 100%
undefined undefined 100%

@Razican
Copy link
Member Author

Razican commented Apr 16, 2020

I think this is ready, but this will require one PR to the gh-pages branch, to retroactively update the benchmark names, since I changed the names of the benchmarks to make them easier to understand and make them more standardized.

@Razican Razican merged commit cb589cd into master Apr 16, 2020
@Razican Razican deleted the new_benchmarks branch April 16, 2020 17:36
@Razican Razican added this to the v0.8.0 milestone Apr 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Benchmarks are failing in master
3 participants