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

Tokenizer: use a def file for keywords #827

Closed
wants to merge 1 commit into from
Closed

Tokenizer: use a def file for keywords #827

wants to merge 1 commit into from

Conversation

Vexu
Copy link
Owner

@Vexu Vexu commented Jan 6, 2025

This should have better performance than StaticStringMap but I haven't tested that yet.

This should have better performance than StaticStringMap
but I haven't tested that yet.
@Vexu
Copy link
Owner Author

Vexu commented Jan 6, 2025

This should have better performance than StaticStringMap but I haven't tested that yet.

$ hyperfine -i -w 1 -r 15 "def/bin/arocc compiler_rt.c -fsyntax-only -I ../zig/lib/" "master/bin/arocc compiler_rt.c -fsyntax-only -I ../zig/lib/"
Benchmark 1: def/bin/arocc compiler_rt.c -fsyntax-only -I ../zig/lib/
  Time (mean ± σ):     230.2 ms ±   3.7 ms    [User: 217.5 ms, System: 11.8 ms]
  Range (min … max):   225.5 ms … 241.3 ms    15 runs
 
  Warning: Ignoring non-zero exit code.
 
Benchmark 2: master/bin/arocc compiler_rt.c -fsyntax-only -I ../zig/lib/
  Time (mean ± σ):     218.4 ms ±   4.1 ms    [User: 204.9 ms, System: 12.5 ms]
  Range (min … max):   212.7 ms … 229.1 ms    15 runs
 
  Warning: Ignoring non-zero exit code.
 
Summary
  master/bin/arocc compiler_rt.c -fsyntax-only -I ../zig/lib/ ran
    1.05 ± 0.03 times faster than def/bin/arocc compiler_rt.c -fsyntax-only -I ../zig/lib/

I guess not.

@Vexu Vexu closed this Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant