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

Quick Fix for compiling issues with icc and OpenMP #489

Merged

Conversation

MSallermann
Copy link
Member

Added a quick workaround to fix compiling issues when using OpenMP and intel compilers. It is pretty much the same that was done in 19a2bdc.
Unit tests seem to run fine on icc 18.03 and also on gcc 6.2.0

- added a workaround for compatibility when using OpenMP and intel compilers
@MSallermann MSallermann changed the title Quick Fix for compiling issues with icc an OpenMP Quick Fix for compiling issues with icc and OpenMP Jan 15, 2019
@coveralls
Copy link

Coverage Status

Coverage remained the same at 84.551% when pulling 0b0250c on MSallermann:feature_icc_compatibility into c5dda64 on spirit-code:develop.

@codecov
Copy link

codecov bot commented Jan 15, 2019

Codecov Report

Merging #489 into develop will increase coverage by 0.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #489      +/-   ##
===========================================
+ Coverage    51.81%   51.83%   +0.01%     
===========================================
  Files           90       90              
  Lines         9844     9848       +4     
===========================================
+ Hits          5101     5105       +4     
  Misses        4743     4743

Copy link
Member

@GPMueller GPMueller left a comment

Choose a reason for hiding this comment

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

Looks good.

@GPMueller GPMueller merged commit c22d4f7 into spirit-code:develop Jan 17, 2019
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