# HG changeset patch # User Doug Simon # Date 1363698450 -3600 # Node ID 8448cbac78e4ca89be5557425e33c3255a0e4537 # Parent 99a71b8c1c5d80a7ef43f04b692a4ec4698f6ed9 made frame omission test ignore padding instructions at the end of installed code diff -r 99a71b8c1c5d -r 8448cbac78e4 graal/com.oracle.graal.hotspot.amd64.test/src/com/oracle/graal/hotspot/amd64/test/AMD64HotSpotFrameOmissionTest.java --- a/graal/com.oracle.graal.hotspot.amd64.test/src/com/oracle/graal/hotspot/amd64/test/AMD64HotSpotFrameOmissionTest.java Tue Mar 19 14:05:40 2013 +0100 +++ b/graal/com.oracle.graal.hotspot.amd64.test/src/com/oracle/graal/hotspot/amd64/test/AMD64HotSpotFrameOmissionTest.java Tue Mar 19 14:07:30 2013 +0100 @@ -25,6 +25,7 @@ import static com.oracle.graal.amd64.AMD64.*; import java.lang.reflect.*; +import java.util.*; import org.junit.*; @@ -109,12 +110,12 @@ for (int i = 0; i < GraalOptions.MethodEndBreakpointGuards; ++i) { asm.int3(); } - while ((asm.codeBuffer.position() % 8) != 0) { - asm.hlt(); - } byte[] expectedCode = asm.codeBuffer.close(true); - byte[] actualCode = installedCode.getCode(); + + // Only compare up to expectedCode.length bytes to ignore + // padding instructions adding during code installation + byte[] actualCode = Arrays.copyOf(installedCode.getCode(), expectedCode.length); Assert.assertArrayEquals(expectedCode, actualCode); }