Replaced literals in nbnxn lists by ints
commit971335dcc39f11cc2e989bdfaf42af446de47a2d
authorBerk Hess <hess@kth.se>
Fri, 22 Jan 2016 10:51:26 +0000 (22 11:51 +0100)
committerGerrit Code Review <gerrit@gerrit.gromacs.org>
Wed, 3 Feb 2016 16:25:56 +0000 (3 17:25 +0100)
treecb14067c45bf772ed55d8535ea54e91b6ce20b58
parent56c79b3a59b8a0e2a633daac0e7cd78cac569cb6
Replaced literals in nbnxn lists by ints

Several literals used for array sizes in nbnxn_pairlist.h and
nbnxn_search.cpp have been replaced by static const int.
The GPU code should assert that these values match the warp size.
Removed the incorrectly used macro NBNXN_PBB_SIMD4.
Removed unused parameter na_c from icell_set_x and in_range functions.
TODOS:
* De-capitalize ints converted from macros in CUDA code.
* Consider replacing macros by ints in OpenCL code.

Change-Id: I467e7ac32c45b4208bfeeef6d5c031e32873f4ce
15 files changed:
src/gromacs/mdlib/nbnxn_atomdata.cpp
src/gromacs/mdlib/nbnxn_consts.h
src/gromacs/mdlib/nbnxn_cuda/nbnxn_cuda.cu
src/gromacs/mdlib/nbnxn_cuda/nbnxn_cuda_kernel.cuh
src/gromacs/mdlib/nbnxn_cuda/nbnxn_cuda_types.h
src/gromacs/mdlib/nbnxn_grid.cpp
src/gromacs/mdlib/nbnxn_internal.h
src/gromacs/mdlib/nbnxn_kernels/nbnxn_kernel_gpu_ref.cpp
src/gromacs/mdlib/nbnxn_ocl/nbnxn_ocl.cpp
src/gromacs/mdlib/nbnxn_ocl/nbnxn_ocl_jit_support.cpp
src/gromacs/mdlib/nbnxn_pairlist.h
src/gromacs/mdlib/nbnxn_search.cpp
src/gromacs/mdlib/nbnxn_search_simd_2xnn.h
src/gromacs/mdlib/nbnxn_search_simd_4xn.h
src/gromacs/mdlib/nbnxn_util.h