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

A straightforward translation of Fukushima's code for elliptic integrals #2200

Merged
merged 12 commits into from
Jun 14, 2019

Conversation

pleroy
Copy link
Member

@pleroy pleroy commented Jun 12, 2019

Complete with a unit test and a benchmark.

This code leaves a lot to be desired, and will need extensive restructuring, but it's useful to have a baseline that is as close as possible to the original FORTRAN.

Run on (4 X 3310 MHz CPU s)
CPU Caches:
  L1 Data 32K (x4)
  L1 Instruction 32K (x4)
  L2 Unified 262K (x4)
  L3 Unified 6291K (x1)
---------------------------------------------------------
Benchmark                  Time           CPU Iterations
---------------------------------------------------------
BM_Elbdj                 412 ns        411 ns    6648000
BM_Elbdj                 411 ns        411 ns    6648000
BM_Elbdj                 413 ns        413 ns    6648000
BM_Elbdj                 412 ns        408 ns    6648000
BM_Elbdj                 413 ns        413 ns    6648000
BM_Elbdj                 418 ns        415 ns    6648000
BM_Elbdj                 423 ns        420 ns    6648000
BM_Elbdj                 418 ns        418 ns    6648000
BM_Elbdj                 415 ns        411 ns    6648000
BM_Elbdj                 433 ns        415 ns    6648000
BM_Elbdj_mean            417 ns        413 ns         10
BM_Elbdj_median          414 ns        413 ns         10
BM_Elbdj_stddev            7 ns          4 ns         10

@eggrobin eggrobin added the LGTM label Jun 14, 2019
@pleroy pleroy merged commit 28845df into mockingbirdnest:master Jun 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants