From: Mark Abraham <mark.j.abraham@gmail.com>
Date: Mon, 19 Jan 2015 10:55:35 +0000 (+0100)
Subject: Fix use of simd.h
X-Git-Url: http://biod.pnpi.spb.ru/gitweb/?a=commitdiff_plain;h=3df2b06e2896161e5f501950b6a065e2375559e2;p=alexxy%2Fgromacs.git

Fix use of simd.h

TPI correctness check was inactive.

Use of SIMD preprocessor symbols in forcerec.c was OK because it got
simd.h from nbnxn_simd.h, but also fixed the erroneous use of
transitive inclusion of simd.h just in case that helps someone.

Fixes #1673

Change-Id: Iaf42a4ec420139485de3e509662ca0892fa662c5
---

diff --git a/src/gromacs/mdlib/forcerec.c b/src/gromacs/mdlib/forcerec.c
index 401f04c042..1ec1c2b171 100644
--- a/src/gromacs/mdlib/forcerec.c
+++ b/src/gromacs/mdlib/forcerec.c
@@ -3,7 +3,7 @@
  *
  * Copyright (c) 1991-2000, University of Groningen, The Netherlands.
  * Copyright (c) 2001-2004, The GROMACS development team.
- * Copyright (c) 2013,2014, by the GROMACS development team, led by
+ * Copyright (c) 2013,2014,2015, by the GROMACS development team, led by
  * Mark Abraham, David van der Spoel, Berk Hess, and Erik Lindahl,
  * and including many others, as listed in the AUTHORS file in the
  * top-level source directory and at http://www.gromacs.org.
@@ -75,6 +75,7 @@
 #include "gmx_omp_nthreads.h"
 #include "gmx_detect_hardware.h"
 #include "inputrec.h"
+#include "gromacs/simd/simd.h"
 
 #include "types/nbnxn_cuda_types_ext.h"
 #include "gpu_utils.h"
diff --git a/src/gromacs/mdlib/tpi.c b/src/gromacs/mdlib/tpi.c
index 8eaa88b740..5a584281e0 100644
--- a/src/gromacs/mdlib/tpi.c
+++ b/src/gromacs/mdlib/tpi.c
@@ -3,7 +3,7 @@
  *
  * Copyright (c) 1991-2000, University of Groningen, The Netherlands.
  * Copyright (c) 2001-2004, The GROMACS development team.
- * Copyright (c) 2013,2014, by the GROMACS development team, led by
+ * Copyright (c) 2013,2014,2015 by the GROMACS development team, led by
  * Mark Abraham, David van der Spoel, Berk Hess, and Erik Lindahl,
  * and including many others, as listed in the AUTHORS file in the
  * top-level source directory and at http://www.gromacs.org.
@@ -70,6 +70,7 @@
 #include "mtop_util.h"
 #include "pme.h"
 #include "gromacs/gmxlib/conformation-utilities.h"
+#include "gromacs/simd/simd.h"
 
 #include "gromacs/fileio/confio.h"
 #include "gromacs/fileio/gmxfio.h"
@@ -672,7 +673,7 @@ double do_tpi(FILE *fplog, t_commrec *cr,
             bEnergyOutOfBounds = FALSE;
 #ifdef GMX_SIMD_X86_SSE2_OR_HIGHER
             /* With SSE the energy can overflow, check for this */
-            if (gmx_mm_check_and_reset_overflow())
+            if (gmx_simd_check_and_reset_overflow())
             {
                 if (debug)
                 {