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

Builtins: handle _Float16 type specifier #594

Merged
merged 1 commit into from
Dec 3, 2023
Merged

Conversation

ehaas
Copy link
Collaborator

@ehaas ehaas commented Dec 2, 2023

With this + #591 + #592 I'm able to parse zig2.c on my linux system without using an external preprocessor 🤯

Current numbers for reference, looks like we need to cut back on memory usage a bit but speed is looking good :)

Benchmark 1 (6 runs): clang -std=c11 -Ilib -fsyntax-only build/zig2.c
  measurement          mean ± σ            min … max           outliers         delta
  wall_time          10.9s  ± 24.4ms    10.9s  … 10.9s           0 ( 0%)        0%
  peak_rss           1.00GB ± 37.0KB    1.00GB … 1.00GB          0 ( 0%)        0%
  cpu_cycles         44.3G  ± 99.6M     44.1G  … 44.4G           0 ( 0%)        0%
  instructions       72.1G  ±  129M     72.0G  … 72.3G           0 ( 0%)        0%
  cache_references    204M  ± 3.58M      201M  …  210M           0 ( 0%)        0%
  cache_misses       26.1M  ±  142K     25.9M  … 26.4M           0 ( 0%)        0%
  branch_misses       102M  ±  324K      101M  …  102M           0 ( 0%)        0%
Benchmark 2 (8 runs): arocc -Wno-integer-overflow -Wno-unused-value -Wno-return-type -std=c11 -I/usr/lib/gcc/x86_64-linux-gnu/10/include -Ilib -fsyntax-only build/zig2.c
  measurement          mean ± σ            min … max           outliers         delta
  wall_time          8.46s  ±  227ms    8.23s  … 8.92s           0 ( 0%)        ⚡- 22.3% ±  1.9%
  peak_rss           1.37GB ± 37.4KB    1.37GB … 1.37GB          0 ( 0%)        💩+ 36.5% ±  0.0%
  cpu_cycles         29.9G  ± 93.9M     29.8G  … 30.1G           0 ( 0%)        ⚡- 32.4% ±  0.3%
  instructions       63.4G  ±  827      63.4G  … 63.4G           0 ( 0%)        ⚡- 12.1% ±  0.1%
  cache_references   78.6M  ±  914K     77.1M  … 80.0M           0 ( 0%)        ⚡- 61.4% ±  1.4%
  cache_misses       35.5M  ±  482K     35.2M  … 36.7M           1 (13%)        💩+ 36.2% ±  1.7%
  branch_misses       289M  ±  333K      288M  …  289M           0 ( 0%)        💩+184.3% ±  0.4%

@Vexu Vexu merged commit a0555b0 into Vexu:master Dec 3, 2023
3 checks passed
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.

2 participants