Skip to content

Commit 5ed3990

Browse files
committed
fixup! Prepare for OpenACC porting in mpas_reconstruct_2d
Fix spacing and whitespace for review comments
1 parent 49336d8 commit 5ed3990

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

src/operators/mpas_vector_reconstruction.F

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -218,7 +218,7 @@ subroutine mpas_reconstruct_2d(meshPool, u, uReconstructX, uReconstructY, uRecon
218218
! temporary arrays needed in the compute procedure
219219
logical :: includeHalosLocal
220220
integer, pointer :: nCells_ptr, nVertLevels_ptr
221-
integer :: nCells, nVertLevels
221+
integer :: nCells, nVertLevels
222222
integer, dimension(:,:), pointer :: edgesOnCell
223223
integer, dimension(:), pointer :: nEdgesOnCell
224224
integer :: iCell,iEdge, i, k
@@ -249,7 +249,7 @@ subroutine mpas_reconstruct_2d(meshPool, u, uReconstructX, uReconstructY, uRecon
249249
call mpas_pool_get_dimension(meshPool, 'nCellsSolve', nCells_ptr)
250250
end if
251251
call mpas_pool_get_dimension(meshPool, 'nVertLevels', nVertLevels_ptr)
252-
! Dereference pointers so that OpenACC copies them correctly
252+
! Dereference pointers to make sure OpenACC copies the value pointed to implicitly
253253
nCells = nCells_ptr
254254
nVertLevels = nVertLevels_ptr
255255

@@ -275,7 +275,7 @@ subroutine mpas_reconstruct_2d(meshPool, u, uReconstructX, uReconstructY, uRecon
275275
do iCell = 1, nCells
276276
! initialize the reconstructed vectors
277277
!$acc loop vector
278-
do k=1,nVertLevels
278+
do k = 1,nVertLevels
279279
uReconstructX(k,iCell) = 0.0
280280
uReconstructY(k,iCell) = 0.0
281281
uReconstructZ(k,iCell) = 0.0
@@ -284,10 +284,10 @@ subroutine mpas_reconstruct_2d(meshPool, u, uReconstructX, uReconstructY, uRecon
284284
! a more efficient reconstruction where rbf_values*matrix_reconstruct
285285
! has been precomputed in coeffs_reconstruct
286286
!$acc loop seq
287-
do i=1,nEdgesOnCell(iCell)
287+
do i = 1,nEdgesOnCell(iCell)
288288
iEdge = edgesOnCell(i,iCell)
289289
!$acc loop vector
290-
do k=1,nVertLevels
290+
do k = 1,nVertLevels
291291
uReconstructX(k,iCell) = uReconstructX(k,iCell) &
292292
+ coeffs_reconstruct(1,i,iCell) * u(k,iEdge)
293293
uReconstructY(k,iCell) = uReconstructY(k,iCell) &
@@ -313,7 +313,7 @@ subroutine mpas_reconstruct_2d(meshPool, u, uReconstructX, uReconstructY, uRecon
313313
clon = cos(lonCell(iCell))
314314
slon = sin(lonCell(iCell))
315315
!$acc loop vector
316-
do k=1,nVertLevels
316+
do k = 1,nVertLevels
317317
uReconstructZonal(k,iCell) = -uReconstructX(k,iCell)*slon + &
318318
uReconstructY(k,iCell)*clon
319319
uReconstructMeridional(k,iCell) = -(uReconstructX(k,iCell)*clon &
@@ -328,7 +328,7 @@ subroutine mpas_reconstruct_2d(meshPool, u, uReconstructX, uReconstructY, uRecon
328328
!$acc parallel default(present)
329329
!$acc loop gang vector collapse(2)
330330
do iCell = 1, nCells
331-
do k=1,nVertLevels
331+
do k = 1,nVertLevels
332332
uReconstructZonal (k,iCell) = uReconstructX(k,iCell)
333333
uReconstructMeridional(k,iCell) = uReconstructY(k,iCell)
334334
end do

0 commit comments

Comments
 (0)