StreamComputing/gromacs — pull request #82 by mabraham (2014-12-09 01:03:12+00:00)
Title: Remove lots of high-level duplication

src/gromacs/mdlib/nbnxn_[cuda,ocl]/nbnxn_[cuda,ocl].h is now
src/gromacs mdlib/nbnxn_gpu.h
(and similarly for data_mgmt.h files)

Renamed some struct members and function names to be neutral with
resepct to GPU flavour. Introduced some typedefs so that we can use
generic names at higher level and specific names at lower level. Not
sure if this will work in the long run, e.g. with doxygen.

Minor enhancements to macro machinery for supporting declaring
the three functions.

The above moves the code much closer to "three implementations of the
same interface," which we need in order to make a reasonable judgement
for whether there is duplication at a lower level that we should
remove. Further work of this type probably warranted in gpu_utils.*

Removed src/gromacs/legacyheaders/types/*ext.h in favour of
explicit forward declarations of structs.

Renamed various data structures along the lines of new Gromacs
standard for C-style structs (ie. struct gmx_some_name_t with no fancy
typedefs hiding whether types are structs or pointers to structs).

Moved GPU timing structs to timing module