Simplify PME GPU constants
commit3f1cf35016a98ce29800b4471d5e8bf28a4bee1c
authorMark Abraham <mark.j.abraham@gmail.com>
Sun, 8 Mar 2020 07:46:08 +0000 (8 08:46 +0100)
committerSzilárd Páll <pall.szilard@gmail.com>
Wed, 18 Mar 2020 14:39:13 +0000 (18 15:39 +0100)
tree3bac35ecf507561b6bd1bd715d52c140eec295d4
parent7bbc3e8a50cb9d3afeb7505c198df7d84606e25b
Simplify PME GPU constants

The values dependent on threadsPerAtom are now computed directly
from a class enum value, rather than indirectly from a bool.

Turned clang-format off for sections of the code where we declare
template functions. That stuff is easier to read and maintain if we
treat it like tabular data rather than free-form code.

Removed some const on template values. Those are useless, because
template values are always const, and confused a compiler when a const
ThreadsPerAtom was compared with a ThreadsPerAtom.

Change-Id: I295d4c2ea52b7912b8bd9a09ee178d104e9bfcb0
src/gromacs/ewald/pme_gather.cu
src/gromacs/ewald/pme_gpu_calculate_splines.cuh
src/gromacs/ewald/pme_gpu_constants.h
src/gromacs/ewald/pme_gpu_internal.cpp
src/gromacs/ewald/pme_gpu_program_impl.cu
src/gromacs/ewald/pme_gpu_program_impl_ocl.cpp
src/gromacs/ewald/pme_gpu_settings.h
src/gromacs/ewald/pme_spread.cu
src/gromacs/ewald/tests/pmetestcommon.cpp