From a3eee419a139e9ea09a1809ebb01aaea92a243f1 Mon Sep 17 00:00:00 2001 From: Yaser Afshar Date: Fri, 6 Mar 2020 09:01:28 -0600 Subject: [PATCH] minor optimization, avoiding double computation and extra communication --- src/min_cg.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/min_cg.cpp b/src/min_cg.cpp index 9db1fd9799..c164fb55d6 100644 --- a/src/min_cg.cpp +++ b/src/min_cg.cpp @@ -109,11 +109,10 @@ int MinCG::iterate(int maxiter) dotall[1] += fextra[i]*gextra[i]; } - fdotf = 0.0; if (update->ftol > 0.0) { if (normstyle == MAX) fdotf = fnorm_max(); // max force norm else if (normstyle == INF) fdotf = fnorm_inf(); // infinite force norm - else if (normstyle == TWO) fdotf = fnorm_sqr(); // Euclidean force 2-norm + else if (normstyle == TWO) fdotf = dotall[0]; // Euclidean force 2-norm else error->all(FLERR,"Illegal min_modify command"); if (fdotf < update->ftol*update->ftol) return FTOL; }