Text archives Help

---

Author: boulos
Date: Tue Apr  1 14:29:40 2008
New Revision: 2159

Modified:
   trunk/CMake/ConfigCompilerFlags.cmake
Log:
Removing -ffast-math from the optimized compile flags line. It has
been causing massive heartache for the DynBVH approximate build
(results in violently different trees, and depending on things like
inlining results in even more differences)

Removing the flag for bin/manta on a 2.66Ghz Clovertown results in
only a 1.5% slowdown and is a lot safer. If someone's code presents a
monstrous slowdown, I'll figure out how to deal with it.



Modified: trunk/CMake/ConfigCompilerFlags.cmake
==============================================================================
--- trunk/CMake/ConfigCompilerFlags.cmake       (original)
+++ trunk/CMake/ConfigCompilerFlags.cmake       Tue Apr  1 14:29:40 2008
@@ -97,7 +97,16 @@
 ###################
 SET(WARNING_FLAGS "-Wall -Wno-strict-aliasing")
 SET(DEBUG_FLAGS "-O0 -g3")
-SET(RELEASE_FLAGS "-O3 -DNDEBUG -g3 -fgcse-sm -funroll-loops 
-fstrict-aliasing -fsched-interblock -ffast-math -freorder-blocks")
+
+####################
+# NOTE(boulos): We have traditionally included
+# -ffast-math in our optimization line. However, for Manta r2158 this
+# only makes a 1.5 percent difference in framerate on bin/manta and
+# can drastically affect the performance of the DynBVH code by
+# producing an incredibly variable output BVH. I'm removing it as of 
(1-Apr-2008)
+####################
+
+SET(RELEASE_FLAGS "-O3 -DNDEBUG -g3 -fgcse-sm -funroll-loops 
-fstrict-aliasing -fsched-interblock -freorder-blocks")
 IF   (USING_GCC)
   APPEND_TO_STRING(C_FLAGS         ${WARNING_FLAGS})
   APPEND_TO_STRING(C_FLAGS_DEBUG   ${DEBUG_FLAGS})

---

• [Manta] r2159 - trunk/CMake, Solomon Boulos, 04/01/2008

---

Archive powered by MHonArc 2.6.16.