Skip to content

Commit bda6c42

Browse files
committed
Cleanup bytecode verifier
Signed-off-by: Theresa Mammarella <[email protected]>
1 parent 1f1eb8a commit bda6c42

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
#if defined(J9VM_OPT_VALHALLA_STRICT_FIELDS)
475474
BOOLEAN anotherInstanceInitCalled = FALSE;
476475
#endif /* defined(J9VM_OPT_VALHALLA_STRICT_FIELDS) */
@@ -646,8 +645,6 @@ verifyBytecodes (J9BytecodeVerificationData * verifyData)
646645
((U_32)(stackBase - liveStack->stackElements) - 1) : 0;
647646
goto _miscError;
648647
}
649-
/* Reset as the flag is only used for the current bytecode */
650-
isNextStack = FALSE;
651648

652649
/* Format: 8bits action, 4bits type Y, 4bits type X */
653650
type1 = (UDATA) J9JavaBytecodeVerificationTable[bc];
@@ -2389,7 +2386,6 @@ verifyBytecodes (J9BytecodeVerificationData * verifyData)
23892386
/* Load the next stack into the live stack */
23902387
SAVE_STACKTOP(liveStack, stackTop);
23912388
memcpy((UDATA *) liveStack, (UDATA *) currentMapData, verifyData->stackSize);
2392-
isNextStack = TRUE;
23932389
RELOAD_LIVESTACK;
23942390

23952391
/* 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
@@ -281,27 +281,6 @@ typedef struct J9BCVAlloc {
281281
temps[ ( tempIndex - 1 ) ] = BCV_BASE_TYPE_TOP; \
282282
}
283283

284-
#define STORE_TEMP_OBJECT( tempIndex, tempType ) \
285-
STORE_TEMP( tempIndex, tempType );
286-
287-
#define STORE_TEMP_INTEGER( tempIndex ) \
288-
STORE_TEMP( tempIndex, (UDATA) (BCV_BASE_TYPE_INT) );
289-
290-
#define STORE_TEMP_FLOAT( tempIndex ) \
291-
STORE_TEMP( tempIndex, (UDATA) (BCV_BASE_TYPE_FLOAT) );
292-
293-
/* direct write of the TOP avoids the pre-index check */
294-
295-
#define STORE_TEMP_PAIR( tempIndex, tempType ) \
296-
STORE_TEMP( tempIndex, tempType ); \
297-
temps[ ( tempIndex + 1 ) ] = (BCV_BASE_TYPE_TOP);
298-
299-
#define STORE_TEMP_LONG( tempIndex ) \
300-
STORE_TEMP_PAIR( tempIndex, (UDATA) (BCV_BASE_TYPE_LONG) );
301-
302-
#define STORE_TEMP_DOUBLE( tempIndex ) \
303-
STORE_TEMP_PAIR( tempIndex, (UDATA) (BCV_BASE_TYPE_DOUBLE) );
304-
305284
#define RELOAD_LIVESTACK \
306285
stackBase = &(liveStack->stackElements[liveStack->stackBaseIndex]); \
307286
stackTop = &(liveStack->stackElements[liveStack->stackTopIndex]); \

0 commit comments

Comments
 (0)