[kaffe] CVS kaffe (dalibor): Replaced WORD macro by inline function
Kaffe CVS
cvs-commits at kaffe.org
Fri Feb 27 08:34:02 PST 2004
PatchSet 4466
Date: 2004/02/27 16:12:38
Author: dalibor
Branch: HEAD
Tag: (none)
Log:
Replaced WORD macro by inline function
2004-02-27 Dalibor Topic <robilad at kaffe.org>
* kaffe/kaffevm/verify.c
(getWord): New static inline function.
(WORD) Removed.
Replaced use of WORD by getWord.
Members:
ChangeLog:1.2046->1.2047
kaffe/kaffevm/verify.c:1.59->1.60
Index: kaffe/ChangeLog
diff -u kaffe/ChangeLog:1.2046 kaffe/ChangeLog:1.2047
--- kaffe/ChangeLog:1.2046 Thu Feb 26 18:02:47 2004
+++ kaffe/ChangeLog Fri Feb 27 16:12:38 2004
@@ -1,3 +1,10 @@
+2004-02-27 Dalibor Topic <robilad at kaffe.org>
+
+ * kaffe/kaffevm/verify.c
+ (getWord): New static inline function.
+ (WORD) Removed.
+ Replaced use of WORD by getWord.
+
2004-02-26 Michael Koch <konqueror at gmx.de>
* libraries/javalib/gnu/java/nio/DatagramChannelImpl.java:
Index: kaffe/kaffe/kaffevm/verify.c
diff -u kaffe/kaffe/kaffevm/verify.c:1.59 kaffe/kaffe/kaffevm/verify.c:1.60
--- kaffe/kaffe/kaffevm/verify.c:1.59 Mon Feb 23 20:00:44 2004
+++ kaffe/kaffe/kaffevm/verify.c Fri Feb 27 16:12:39 2004
@@ -908,9 +908,16 @@
/* these retrieve the word (16 bits) or double world (32 bits) of bytecode starting
* at pc = _PC
*/
-#define WORD(_CODE, _PC) ((int16)( \
- (_CODE[(_PC)+0] << 8) | \
- (_CODE[(_PC)+1])))
+
+static inline
+int16
+getWord(const unsigned char* code,
+ const unsigned int pc)
+{
+ return ((int16)((code[(pc)+0] << 8)
+ | (code[(pc)+1])));
+}
+
#define DWORD(_CODE, _PC) ((int32)( \
(_CODE[(_PC)+0] << 24) | \
(_CODE[(_PC)+1] << 16) | \
@@ -1405,7 +1412,7 @@
#define GET_WIDX(_IDX, _PC) \
_IDX = (_PC) + 1; \
- _IDX = WORD(code, _IDX); \
+ _IDX = getWord(code, _IDX); \
CHECK_POOL_IDX(_IDX)
#define BRANCH_IN_BOUNDS(_N, _INST) \
@@ -1686,7 +1693,7 @@
pc++;
wide = false;
- n = WORD(code, pc);
+ n = getWord(code, pc);
}
else {
n = code[pc + 1];
@@ -1705,7 +1712,7 @@
pc++;
wide = false;
- n = WORD(code, pc);
+ n = getWord(code, pc);
}
else {
GET_IDX(n, pc);
@@ -1739,7 +1746,7 @@
status[pc] |= END_BLOCK;
n = pc + 1;
- branchoffset = WORD(code, n);
+ branchoffset = getWord(code, n);
newpc = pc + branchoffset;
BRANCH_IN_BOUNDS(newpc, "goto");
status[newpc] |= START_BLOCK;
@@ -1773,7 +1780,7 @@
status[newpc] |= START_BLOCK;
n = pc + 1;
- branchoffset = WORD(code, n);
+ branchoffset = getWord(code, n);
newpc = pc + branchoffset;
BRANCH_IN_BOUNDS(newpc, "if<condition> = true");
status[newpc] |= START_BLOCK;
@@ -1782,7 +1789,7 @@
case JSR:
newpc = pc + 1;
- newpc = pc + WORD(code, newpc);
+ newpc = pc + getWord(code, newpc);
goto JSR_common;
case JSR_W:
newpc = pc + 1;
@@ -2182,7 +2189,7 @@
{
case GOTO:
newpc = pc + 1;
- newpc = pc + WORD(code, newpc);
+ newpc = pc + getWord(code, newpc);
nextBlock = inWhichBlock(newpc, blocks, numBlocks);
if (!merge(einfo, method, curBlock, nextBlock)) {
@@ -2202,7 +2209,7 @@
case JSR:
newpc = pc + 1;
- newpc = pc + WORD(code, newpc);
+ newpc = pc + getWord(code, newpc);
goto JSR_common;
case JSR_W:
newpc = pc + 1;
@@ -2224,7 +2231,7 @@
case RET:
if (status[pc] & WIDE_MODDED) {
n = pc + 1;
- n = WORD(code, n);
+ n = getWord(code, n);
} else {
n = code[pc + 1];
}
@@ -2259,7 +2266,7 @@
case IF_ICMPLT: case IFLT:
case IF_ICMPLE: case IFLE:
newpc = pc + 1;
- newpc = pc + WORD(code, newpc);
+ newpc = pc + getWord(code, newpc);
nextBlock = inWhichBlock(newpc, blocks, numBlocks);
if (!merge(einfo, method, curBlock, nextBlock)) {
@@ -2747,7 +2754,7 @@
idx = code[pc + 1]
#define GET_WIDX \
- idx = pc + 1; idx = WORD(code, idx)
+ idx = pc + 1; idx = getWord(code, idx)
/* checks whether the specified local variable is of the specified type. */
@@ -4128,7 +4135,7 @@
const uint32 opcode = code[pc];
const constants* pool = CLASS_CONSTANTS(method->class);
- const uint32 idx = WORD(code, pc + 1);
+ const uint32 idx = getWord(code, pc + 1);
const uint32 classIdx = METHODREF_CLASS(idx, pool);
Type mrc;
More information about the kaffe
mailing list