MRChemSoft / mrchem

MultiResolution Chemistry
GNU Lesser General Public License v3.0
30 stars 21 forks source link

Update compiler flags #268

Closed stigrj closed 4 years ago

stigrj commented 4 years ago

Added some compiler flags. The missing -xHost Intel flag had a significant effect on performance (30%) and accounts for the discrepancy we saw between Intel and GNU before.

Not sure if all the other should be there (they come from EasyBuild defaults) or if we need more?

codecov[bot] commented 4 years ago

Codecov Report

Merging #268 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #268   +/-   ##
=======================================
  Coverage   54.44%   54.44%           
=======================================
  Files         105      105           
  Lines        8319     8319           
=======================================
  Hits         4529     4529           
  Misses       3790     3790

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 3c39fce...543c67e. Read the comment docs.

robertodr commented 4 years ago

Is CircleCI blocked somehow?