[PATCH] D36738: [test-suite] Adding miniXyce Benchmark

Brian Homerding via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 17 13:21:00 PDT 2017


homerdin added a comment.

In https://reviews.llvm.org/D36738#842616, @MatzeB wrote:

> - On my macOS x86_64 system it compiles warning free but fails to verify: ``` $ diff -u /Users/mbraun/dev/test-suite/build/MultiSource/Benchmarks/DOE-ProxyApps-C++/miniXyce/Output/miniXyce.test.out /Users/mbraun/dev/test-suite/MultiSource/Benchmarks/DOE-ProxyApps-C++/miniXyce/miniXyce.reference_output
>   - /Users/mbraun/dev/test-suite/build/MultiSource/Benchmarks/DOE-ProxyApps-C++/miniXyce/Output/miniXyce.test.out	2017-08-15 14:10:36.000000000 -0700 +++ /Users/mbraun/dev/test-suite/MultiSource/Benchmarks/DOE-ProxyApps-C++/miniXyce/miniXyce.reference_output	2017-08-15 14:09:10.000000000 -0700 @@ -32,5 +32,5 @@ GMRES_tolerance: 1e-06 GMRES_subspace_dim: 10 GMRES_average_iters: 5
> - GMRES_average_res: 5.65781e-16 +  GMRES_average_res: 5.79223e-16 exit 0
>
>   ```


Was able to reproduce this issue on my mac.  We figured out that the issue in from the use of the sin function.

  mX_source.cpp:51:  return (offset + amplitude * sin(2*pi*freq*t + phase));
  mX_source.cpp:67:  return (offset + amplitude * sin(2*pi*carrier_freq*t + modulation_index*sin(2*pi*signal_freq*t)));

I rebuilt the application linking to the JuliaLang openlibm on both machines and am now getting `GMRES_average_res: 5.73829e-16` on both machines.
Still looking into how to approach this issue for the test-suite.

Thanks,
Brian


https://reviews.llvm.org/D36738





More information about the llvm-commits mailing list