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

PWR046: Expand to discuss Fortran and add benchmark #61

Merged
merged 3 commits into from
Dec 24, 2024

Conversation

alvrogd
Copy link
Collaborator

@alvrogd alvrogd commented Dec 20, 2024

This PR expands the PWR046 entry to discuss its applicability to Fortran, and adds benchmarks for both C and Fortran. My results:

$ lscpu
  Model name:             13th Gen Intel(R) Core(TM) i7-13700H

$ gcc --version
gcc (Debian 12.2.0-14) 12.2.0

$ gfortran --version
GNU Fortran (Debian 12.2.0-14) 12.2.0

$ ./run-benchmarks.py --check PWR046
------------------------------------------------------------------
Benchmark                        Time             CPU   Iterations
------------------------------------------------------------------
PWR046 C Example              4165 us         4165 us          168
PWR046 C Improved             3035 us         3035 us          232
PWR046 Fortran Example        4154 us         4154 us          167
PWR046 Fortran Improved       3022 us         3022 us          231

@alvrogd alvrogd self-assigned this Dec 20, 2024
@alvrogd alvrogd force-pushed the feature/AddPWR046Benchmark branch from 804612e to 6d5f58d Compare December 20, 2024 09:40
@alvrogd alvrogd marked this pull request as ready for review December 20, 2024 09:43
@alvrogd alvrogd requested review from a team, ruifm, inaki-amatria and daniel-otero and removed request for a team December 20, 2024 09:43
@alvrogd alvrogd mentioned this pull request Dec 20, 2024
README.md Show resolved Hide resolved
@alvrogd alvrogd requested review from d-alonso and igarcia0 December 23, 2024 08:14
Copy link
Contributor

@daniel-otero daniel-otero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

We need to add a page on the benchmark performance so we don't have to look back on each PR to see a sample.

@alvrogd alvrogd force-pushed the feature/AddPWR046Benchmark branch from 6d5f58d to d03e859 Compare December 24, 2024 08:50
@alvrogd alvrogd merged commit d03e859 into main Dec 24, 2024
7 checks passed
@alvrogd alvrogd deleted the feature/AddPWR046Benchmark branch December 24, 2024 08:53
@alvrogd
Copy link
Collaborator Author

alvrogd commented Dec 24, 2024

We need to add a page on the benchmark performance so we don't have to look back on each PR to see a sample.

That would be a great idea! I take note of that.

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.

3 participants