[cig-commits] r19518 - seismo/3D/SPECFEM3D_GLOBE/trunk/src/specfem3D

[dkomati1 at geodynamics.org]

Author: dkomati1
Date: 2012-01-30 14:06:33 -0800 (Mon, 30 Jan 2012)
New Revision: 19518

Modified:
   seismo/3D/SPECFEM3D_GLOBE/trunk/src/specfem3D/compute_element.F90
   seismo/3D/SPECFEM3D_GLOBE/trunk/src/specfem3D/compute_forces_crust_mantle_Dev.F90
   seismo/3D/SPECFEM3D_GLOBE/trunk/src/specfem3D/compute_forces_inner_core_Dev.F90
   seismo/3D/SPECFEM3D_GLOBE/trunk/src/specfem3D/specfem3D.F90
Log:
added comments / warnings about using _HANDOPT


Modified: seismo/3D/SPECFEM3D_GLOBE/trunk/src/specfem3D/compute_element.F90
===================================================================
--- seismo/3D/SPECFEM3D_GLOBE/trunk/src/specfem3D/compute_element.F90	2012-01-30 21:51:13 UTC (rev 19517)
+++ seismo/3D/SPECFEM3D_GLOBE/trunk/src/specfem3D/compute_element.F90	2012-01-30 22:06:33 UTC (rev 19518)
@@ -30,6 +30,12 @@
 ! or compile with: -D_HANDOPT
 !#define _HANDOPT
 
+! BEWARE:
+! BEWARE: we have observed that using _HANDOPT in combination with -O3 or higher can lead to problems on some machines;
+! BEWARE: thus, be careful when using it. At the very least, run the same test simulation once with _HANDOPT and once without
+! BEWARE: and make sure that all the seismograms you get are the same down to roundoff noise.
+! BEWARE:
+
 ! note: these hand optimizations should help compilers to pipeline the code and make better use of the cache;
 !          depending on compilers, it can further decrease the computation time by ~ 30%.
 !          the original routines are commented with "! way 1", the hand-optimized routines with  "! way 2"

Modified: seismo/3D/SPECFEM3D_GLOBE/trunk/src/specfem3D/compute_forces_crust_mantle_Dev.F90
===================================================================
--- seismo/3D/SPECFEM3D_GLOBE/trunk/src/specfem3D/compute_forces_crust_mantle_Dev.F90	2012-01-30 21:51:13 UTC (rev 19517)
+++ seismo/3D/SPECFEM3D_GLOBE/trunk/src/specfem3D/compute_forces_crust_mantle_Dev.F90	2012-01-30 22:06:33 UTC (rev 19518)
@@ -30,6 +30,12 @@
 ! or compile with: -D_HANDOPT
 !#define _HANDOPT
 
+! BEWARE:
+! BEWARE: we have observed that using _HANDOPT in combination with -O3 or higher can lead to problems on some machines;
+! BEWARE: thus, be careful when using it. At the very least, run the same test simulation once with _HANDOPT and once without
+! BEWARE: and make sure that all the seismograms you get are the same down to roundoff noise.
+! BEWARE:
+
 ! note: these hand optimizations should help compilers to pipeline the code and make better use of the cache;
 !          depending on compilers, it can further decrease the computation time by ~ 30%.
 !          the original routines are commented with "! way 1", the hand-optimized routines with  "! way 2"

Modified: seismo/3D/SPECFEM3D_GLOBE/trunk/src/specfem3D/compute_forces_inner_core_Dev.F90
===================================================================
--- seismo/3D/SPECFEM3D_GLOBE/trunk/src/specfem3D/compute_forces_inner_core_Dev.F90	2012-01-30 21:51:13 UTC (rev 19517)
+++ seismo/3D/SPECFEM3D_GLOBE/trunk/src/specfem3D/compute_forces_inner_core_Dev.F90	2012-01-30 22:06:33 UTC (rev 19518)
@@ -30,6 +30,12 @@
 ! or compile with: -D_HANDOPT
 !#define _HANDOPT
 
+! BEWARE:
+! BEWARE: we have observed that using _HANDOPT in combination with -O3 or higher can lead to problems on some machines;
+! BEWARE: thus, be careful when using it. At the very least, run the same test simulation once with _HANDOPT and once without
+! BEWARE: and make sure that all the seismograms you get are the same down to roundoff noise.
+! BEWARE:
+
 ! note: these hand optimizations should help compilers to pipeline the code and make better use of the cache;
 !          depending on compilers, it can further decrease the computation time by ~ 30%.
 !          the original routines are commented with "! way 1", the hand-optimized routines with  "! way 2"

Modified: seismo/3D/SPECFEM3D_GLOBE/trunk/src/specfem3D/specfem3D.F90
===================================================================
--- seismo/3D/SPECFEM3D_GLOBE/trunk/src/specfem3D/specfem3D.F90	2012-01-30 21:51:13 UTC (rev 19517)
+++ seismo/3D/SPECFEM3D_GLOBE/trunk/src/specfem3D/specfem3D.F90	2012-01-30 22:06:33 UTC (rev 19518)
@@ -34,6 +34,12 @@
 !
 !#define _HANDOPT
 
+! BEWARE:
+! BEWARE: we have observed that using _HANDOPT in combination with -O3 or higher can lead to problems on some machines;
+! BEWARE: thus, be careful when using it. At the very least, run the same test simulation once with _HANDOPT and once without
+! BEWARE: and make sure that all the seismograms you get are the same down to roundoff noise.
+! BEWARE:
+
 ! note: these hand optimizations should help compilers to pipeline the code and make better use of the cache;
 !          depending on compilers, it can further decrease the computation time by ~ 30%.
 !          the original routines are commented with "! way 1", the hand-optimized routines with  "! way 2"