[Qemu-devel] [7105] Add a --enable-debug-tcg option to configure

[Qemu-devel] [7105] Add a --enable-debug-tcg option to configure

[Aurelien Jarno]

Revision: 7105
          http://svn.sv.gnu.org/viewvc/?view=rev&root=qemu&revision=7105
Author:   aurel32
Date:     2009-04-13 18:45:38 +0000 (Mon, 13 Apr 2009)
Log Message:
-----------
Add a --enable-debug-tcg option to configure

This patch allows DEBUG_TCGV to be defined (and also prevents NDEBUG
from being defined) when passing an option to the configure script.
This should help to prevent any accidental changes that enable
DEBUG_TCGV in tcg/tcg.h from being committed in future, and may
help to encourage testing with DEBUG_TCGV enabled.

Signed-off-by: Stuart Brady <stuart.brady@gmail.com>
Signed-off-by: Aurelien Jarno <aurelien@aurel32.net>

Modified Paths:
--------------
    trunk/configure
    trunk/tcg/tcg.c
    trunk/tcg/tcg.h

Modified: trunk/configure
===================================================================
--- trunk/configure	2009-04-13 17:19:26 UTC (rev 7104)
+++ trunk/configure	2009-04-13 18:45:38 UTC (rev 7105)
@@ -153,6 +153,7 @@
   ;;
 esac
 gprof="no"
+debug_tcg="no"
 sparse="no"
 strip_opt="yes"
 bigendian="no"
@@ -400,6 +401,10 @@
   ;;
   --audio-drv-list=*) audio_drv_list="$optarg"
   ;;
+  --enable-debug-tcg) debug_tcg="yes"
+  ;;
+  --disable-debug-tcg) debug_tcg="no"
+  ;;
   --enable-sparse) sparse="yes"
   ;;
   --disable-sparse) sparse="no"
@@ -568,6 +573,8 @@
 echo "  --make=MAKE              use specified make [$make]"
 echo "  --install=INSTALL        use specified install [$install]"
 echo "  --static                 enable static build [$static]"
+echo "  --enable-debug-tcg       enable TCG debugging"
+echo "  --disable-debug-tcg      disable TCG debugging (default)"
 echo "  --enable-sparse          enable sparse checker"
 echo "  --disable-sparse         disable sparse checker (default)"
 echo "  --disable-strip          disable stripping binaries"
@@ -1368,6 +1375,9 @@
     exit 1
   ;;
 esac
+if test "$debug_tcg" = "yes" ; then
+  echo "#define DEBUG_TCG 1" >> $config_h
+fi
 if test "$sparse" = "yes" ; then
   echo "CC      := REAL_CC=\"\$(CC)\" cgcc"       >> $config_mak
   echo "HOST_CC := REAL_CC=\"\$(HOST_CC)\" cgcc"  >> $config_mak

Modified: trunk/tcg/tcg.c
===================================================================
--- trunk/tcg/tcg.c	2009-04-13 17:19:26 UTC (rev 7104)
+++ trunk/tcg/tcg.c	2009-04-13 18:45:38 UTC (rev 7105)
@@ -22,9 +22,6 @@
  * THE SOFTWARE.
  */
 
-/* define it to suppress various consistency checks (faster) */
-#define NDEBUG
-
 /* define it to use liveness analysis (better code) */
 #define USE_LIVENESS_ANALYSIS
 
@@ -45,6 +42,11 @@
 #include "qemu-common.h"
 #include "cache-utils.h"
 
+#ifndef DEBUG_TCG
+/* define it to suppress various consistency checks (faster) */
+#define NDEBUG
+#endif
+
 /* Note: the long term plan is to reduce the dependancies on the QEMU
    CPU definitions. Currently they are used for qemu_ld/st
    instructions */

Modified: trunk/tcg/tcg.h
===================================================================
--- trunk/tcg/tcg.h	2009-04-13 17:19:26 UTC (rev 7104)
+++ trunk/tcg/tcg.h	2009-04-13 18:45:38 UTC (rev 7105)
@@ -21,6 +21,7 @@
  * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
  * THE SOFTWARE.
  */
+#include "qemu-common.h"
 #include "tcg-target.h"
 
 #if TCG_TARGET_REG_BITS == 32
@@ -120,7 +121,9 @@
    are aliases for target_ulong and host pointer sized values respectively.
  */
 
-//#define DEBUG_TCGV 1
+#ifdef DEBUG_TCG
+#define DEBUG_TCGV 1
+#endif
 
 #ifdef DEBUG_TCGV