Skip to content

Commit 255bc73

Browse files
committed
Cleanup bytecode verifier
Signed-off-by: Theresa Mammarella <[email protected]>
1 parent 152bbb2 commit 255bc73

File tree

2 files changed

+0
-25
lines changed

2 files changed

+0
-25
lines changed

runtime/bcverify/rtverify.c

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -470,7 +470,6 @@ verifyBytecodes (J9BytecodeVerificationData * verifyData)
470470
UDATA errorTargetType = (UDATA)-1;
471471
UDATA errorStackIndex = (UDATA)-1;
472472
UDATA errorTempData = (UDATA)-1;
473-
BOOLEAN isNextStack = FALSE;
474473

475474
Trc_RTV_verifyBytecodes_Entry(verifyData->vmStruct,
476475
(UDATA) J9UTF8_LENGTH(J9ROMMETHOD_NAME(romMethod)),
@@ -643,8 +642,6 @@ verifyBytecodes (J9BytecodeVerificationData * verifyData)
643642
((U_32)(stackBase - liveStack->stackElements) - 1) : 0;
644643
goto _miscError;
645644
}
646-
/* Reset as the flag is only used for the current bytecode */
647-
isNextStack = FALSE;
648645

649646
/* Format: 8bits action, 4bits type Y, 4bits type X */
650647
type1 = (UDATA) J9JavaBytecodeVerificationTable[bc];
@@ -2365,7 +2362,6 @@ verifyBytecodes (J9BytecodeVerificationData * verifyData)
23652362
/* Load the next stack into the live stack */
23662363
SAVE_STACKTOP(liveStack, stackTop);
23672364
memcpy((UDATA *) liveStack, (UDATA *) currentMapData, verifyData->stackSize);
2368-
isNextStack = TRUE;
23692365
RELOAD_LIVESTACK;
23702366

23712367
/* Start with uninitialized_this flag for the current map data */

runtime/oti/bcverify.h

Lines changed: 0 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -273,27 +273,6 @@ typedef struct J9BCVAlloc {
273273
temps[ ( tempIndex - 1 ) ] = BCV_BASE_TYPE_TOP; \
274274
}
275275

276-
#define STORE_TEMP_OBJECT( tempIndex, tempType ) \
277-
STORE_TEMP( tempIndex, tempType );
278-
279-
#define STORE_TEMP_INTEGER( tempIndex ) \
280-
STORE_TEMP( tempIndex, (UDATA) (BCV_BASE_TYPE_INT) );
281-
282-
#define STORE_TEMP_FLOAT( tempIndex ) \
283-
STORE_TEMP( tempIndex, (UDATA) (BCV_BASE_TYPE_FLOAT) );
284-
285-
/* direct write of the TOP avoids the pre-index check */
286-
287-
#define STORE_TEMP_PAIR( tempIndex, tempType ) \
288-
STORE_TEMP( tempIndex, tempType ); \
289-
temps[ ( tempIndex + 1 ) ] = (BCV_BASE_TYPE_TOP);
290-
291-
#define STORE_TEMP_LONG( tempIndex ) \
292-
STORE_TEMP_PAIR( tempIndex, (UDATA) (BCV_BASE_TYPE_LONG) );
293-
294-
#define STORE_TEMP_DOUBLE( tempIndex ) \
295-
STORE_TEMP_PAIR( tempIndex, (UDATA) (BCV_BASE_TYPE_DOUBLE) );
296-
297276
#define RELOAD_LIVESTACK \
298277
stackBase = &(liveStack->stackElements[liveStack->stackBaseIndex]); \
299278
stackTop = &(liveStack->stackElements[liveStack->stackTopIndex]); \

0 commit comments

Comments
 (0)