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

bench: Implement luau-analyze and luau --compile benchmarks #575

Merged
merged 4 commits into from
Jul 5, 2022
Merged

Conversation

zeux
Copy link
Collaborator

@zeux zeux commented Jul 5, 2022

This change adds another file for benchmarking luau-analyze and sets up
benchmarks for both non-strict/strict modes for analysis and all three
optimization levels for compilation performance.

To avoid issues with race conditions on repository update we do all this
in the same job in benchmark.yml.

To be able to benchmark both modes from a single file, luau-analyze
gains --mode argument which allows to override the default typechecking
mode. Not sure if we'll want this to be a hard override on top of the
module-specified mode in the future, but this works for now.

Benchmark pages with pre-filled data:
https://perf.luau-lang.org/?analyze
https://perf.luau-lang.org/?compile

zeux added 2 commits July 4, 2022 19:04
This change adds another file for benchmarking luau-analyze and sets up
benchmarks for both non-strict/strict modes for analysis and all three
optimization levels for compilation performance.

To avoid issues with race conditions on repository update we do all this
in the same job in benchmark.yml.

To be able to benchmark both modes from a single file, luau-analyze
gains --mode argument which allows to override the default typechecking
mode. Not sure if we'll want this to be a hard override on top of the
module-specified mode in the future, but this works for now.
@zeux zeux changed the title ench: Implement luau-analyze and luau --compile benchmarks bench: Implement luau-analyze and luau --compile benchmarks Jul 5, 2022
zeux added 2 commits July 4, 2022 19:24
This makes sure we just measure parse+typeck+lint time.
@zeux zeux merged commit 2460e38 into master Jul 5, 2022
@zeux zeux deleted the benchmark branch July 5, 2022 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants