Skip to content

Commit

Permalink
Revert "Revert "Merge branch 'mt5555/homme/intel17bug-workaround' (PR #…
Browse files Browse the repository at this point in the history
…1922)""

This reverts commit e88e39a.

The restores the bug fix for Intel17

[non-BFB] for B,F cases on Intel17
  • Loading branch information
rljacob committed Jan 29, 2018
1 parent 8803139 commit 2e145ac
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions components/homme/src/preqx/share/prim_advance_mod.F90
Original file line number Diff line number Diff line change
Expand Up @@ -942,8 +942,13 @@ subroutine advance_hypervis_dp(edge3,elem,hvcoord,hybrid,deriv,nt,nets,nete,dt2,
do j=1,np
do i=1,np
! update v first (gives better results than updating v after heating)
elem(ie)%state%v(i,j,:,k,nt)=elem(ie)%state%v(i,j,:,k,nt) + &
vtens(i,j,:,k,ie)
! intel/17 compiler bug on Edison produces incorrect results with this:
!elem(ie)%state%v(i,j,:,k,nt)=elem(ie)%state%v(i,j,:,k,nt) + &
! vtens(i,j,:,k,ie)
elem(ie)%state%v(i,j,1,k,nt)=elem(ie)%state%v(i,j,1,k,nt) + &
vtens(i,j,1,k,ie)
elem(ie)%state%v(i,j,2,k,nt)=elem(ie)%state%v(i,j,2,k,nt) + &
vtens(i,j,2,k,ie)

v1=elem(ie)%state%v(i,j,1,k,nt)
v2=elem(ie)%state%v(i,j,2,k,nt)
Expand Down

0 comments on commit 2e145ac

Please sign in to comment.