[kaffe] CVS kaffe (doogie): With recent DBG() macro now behaving like a real statement, all calls to it

Kaffe CVS cvs-commits at kaffe.org
Sun Dec 19 15:49:24 PST 2004


PatchSet 5670 
Date: 2004/12/19 23:45:06
Author: doogie
Branch: HEAD
Tag: (none) 
Log:
With recent DBG() macro now behaving like a real statement, all calls to it
must be followed with a ;.  Since the above files weren't the default on my
machine, they got missed in the cleanup.

Members: 
	ChangeLog:1.3215->1.3216 
	config/i386/jit-i386.def:1.17->1.18 
	kaffe/kaffevm/intrp/machine.c:1.43->1.44 
	kaffe/kaffevm/jit/machine.c:1.70->1.71 

Index: kaffe/ChangeLog
diff -u kaffe/ChangeLog:1.3215 kaffe/ChangeLog:1.3216
--- kaffe/ChangeLog:1.3215	Sun Dec 19 23:12:22 2004
+++ kaffe/ChangeLog	Sun Dec 19 23:45:06 2004
@@ -1,5 +1,13 @@
 2004-12-19  Adam Heath  <doogie at brainfood.com>
 
+	* config/i386/jit-i386.def, kaffe/kaffevm/intrp/machine.c,
+	  kaffe/kaffevm/jit/machine.c:
+	  With recent DBG() macro now behaving like a real statement, all
+	  calls to it must be followed with a ;.  Since the above files weren't
+	  the default on my machine, they got missed in the cleanup.
+
+2004-12-19  Adam Heath  <doogie at brainfood.com>
+
 	* kaffe/kaffevm/kaffe.def:
 	  Make the trace_jcode macro take variable args.
 
Index: kaffe/config/i386/jit-i386.def
diff -u kaffe/config/i386/jit-i386.def:1.17 kaffe/config/i386/jit-i386.def:1.18
--- kaffe/config/i386/jit-i386.def:1.17	Tue Jul 13 14:17:03 2004
+++ kaffe/config/i386/jit-i386.def	Sun Dec 19 23:45:09 2004
@@ -1697,7 +1697,7 @@
 {
 DBG(REGFORCE,
     dprintf ("kill_readonce_register(%s)\n", regname(s->regno));
-    )
+    );
 	if (((reginfo[s->regno].ctype & Rdouble) && (s->regno == REG_dbl0)) ||
 	    ((reginfo[s->regno].ctype & Rfloat) && (s->regno == REG_flt0))) {
 	    	OUT(0xDD);
Index: kaffe/kaffe/kaffevm/intrp/machine.c
diff -u kaffe/kaffe/kaffevm/intrp/machine.c:1.43 kaffe/kaffe/kaffevm/intrp/machine.c:1.44
--- kaffe/kaffe/kaffevm/intrp/machine.c:1.43	Thu Dec 16 06:13:52 2004
+++ kaffe/kaffe/kaffevm/intrp/machine.c	Sun Dec 19 23:45:09 2004
@@ -56,12 +56,12 @@
 
 #define	define_insn(code)	break;					\
 				case code:				\
-				IDBG( dprintf("%03ld: %s\n", (long) pc, #code); )
+				IDBG( dprintf("%03ld: %s\n", (long) pc, #code); );
 #define	define_insn_alias(code)	case code:				\
-				IDBG( dprintf("%03ld: %s\n", (long) pc, #code); )
+				IDBG( dprintf("%03ld: %s\n", (long) pc, #code); );
 #define	define_wide_insn(code)	break;					\
 				case code:				\
-				IDBG( dprintf("%03ld: %s\n", (long) pc, #code); )
+				IDBG( dprintf("%03ld: %s\n", (long) pc, #code); );
 
 #define EXPLICIT_CHECK_NULL(_i, _s, _n)                       \
       cbranch_ref_const_ne((_s), 0, reference_label(_i, _n)); \
@@ -127,12 +127,12 @@
 		}
 	}
 
-CDBG(	dprintf("Call: %s.%s%s.\n", meth->class->name->data, meth->name->data, METHOD_SIGD(meth)); )
+CDBG(	dprintf("Call: %s.%s%s.\n", meth->class->name->data, meth->name->data, METHOD_SIGD(meth)); );
 
 	/* If this is native, then call the real function */
 	methaccflags = meth->accflags;
 	if (methaccflags & ACC_NATIVE) {
-NDBG(		dprintf("Call to native %s.%s%s.\n", meth->class->name->data, meth->name->data, METHOD_SIGD(meth)); )
+NDBG(		dprintf("Call to native %s.%s%s.\n", meth->class->name->data, meth->name->data, METHOD_SIGD(meth)); );
 		if (methaccflags & ACC_STATIC) {
 			callMethodA(meth, meth, 0, (jvalue*)arg, (jvalue*)retval, 1);
 		}
@@ -235,7 +235,7 @@
 
 	cleanupExceptionHandling(&mjbuf, thread_data);
 
-RDBG(	dprintf("Returning from method %s%s.\n", meth->name->data, METHOD_SIGD(meth)); )
+RDBG(	dprintf("Returning from method %s%s.\n", meth->name->data, METHOD_SIGD(meth)); );
 }
 
 void runVirtualMachine(methods *meth, slots *lcl, slots *sp, uintp npc, slots *retval, volatile VmExceptHandler *mjbuf, threadData *thread_data) {
Index: kaffe/kaffe/kaffevm/jit/machine.c
diff -u kaffe/kaffe/kaffevm/jit/machine.c:1.70 kaffe/kaffe/kaffevm/jit/machine.c:1.71
--- kaffe/kaffe/kaffevm/jit/machine.c:1.70	Thu Dec 16 06:13:53 2004
+++ kaffe/kaffe/kaffevm/jit/machine.c	Sun Dec 19 23:45:09 2004
@@ -222,7 +222,7 @@
 DBG(MOREJIT,
 	dprintf("asked to translate = %s.%s(%s)\n", 
 	    meth->class->name->data, meth->name->data, METHOD_SIGD(meth));	
-    )
+    );
 
 	/* If this code block is native, then just set it up and return */
 	if ((meth->accflags & ACC_NATIVE) != 0) {
@@ -249,7 +249,7 @@
 DBG(MOREJIT,
 	dprintf("successfully verified = %s.%s(%s)\n", 
 	    meth->class->name->data, meth->name->data, METHOD_SIGD(meth));	
-    )
+    );
 
 	/* Only one in the translator at once. */
 	enterTranslator();
@@ -278,7 +278,7 @@
 			jitting->class->name->data, jitting->name->data, 
 			METHOD_SIGD(jitting));	
 	}
-    )
+    );
 	/* start modifying global variables now */
 	assert(jitting == 0 || !!!"reentered jitter");	/* DEBUG */
 	jitting = meth;					/* DEBUG */
@@ -335,7 +335,7 @@
 			while (npc < len && !IS_STARTOFBASICBLOCK(npc) && !IS_STARTOFEXCEPTION(npc)) {
 				npc = npc + insnLen[base[npc]];
 			}
-DBG(JIT,		dprintf("unreachable basic block pc [%d:%d]\n", pc, npc - 1);   )
+DBG(JIT,		dprintf("unreachable basic block pc [%d:%d]\n", pc, npc - 1);   );
 			if (IS_STARTOFBASICBLOCK(npc)) {
 				end_basic_block();
 				start_basic_block();
@@ -344,7 +344,7 @@
 			continue;
 		}
 
-DBG(JIT,	dprintf("pc = %d, npc = %d\n", pc, npc);	)
+DBG(JIT,	dprintf("pc = %d, npc = %d\n", pc, npc);	);
 
 		/* Determine various exception conditions */
 		checkCaughtExceptions(meth, pc);
@@ -400,7 +400,7 @@
 	dprintf("Translated %s.%s%s (%s) %p\n", meth->class->name->data, 
 		meth->name->data, METHOD_SIGD(meth), 
 		isStatic ? "static" : "normal", METHOD_NATIVECODE(meth));	
-    )
+    );
 
 	if (Kaffe_JavaVMArgs.enableVerboseJIT) {
 		tme = currentTime();



More information about the kaffe mailing list