# HG changeset patch # User Thomas Wuerthinger # Date 1305117800 -7200 # Node ID ccab1d5f345ebbe5ab06d457ed48c42ecc79bac5 # Parent 62306ff0ccc248bf1ce1e66798b72041bb52bd36 Removed TestPatching option. diff -r 62306ff0ccc2 -r ccab1d5f345e graal/GraalCompiler/src/com/sun/c1x/C1XOptions.java --- a/graal/GraalCompiler/src/com/sun/c1x/C1XOptions.java Wed May 11 14:42:19 2011 +0200 +++ b/graal/GraalCompiler/src/com/sun/c1x/C1XOptions.java Wed May 11 14:43:20 2011 +0200 @@ -53,7 +53,6 @@ // debugging settings public static boolean VerifyPointerMaps = ____; public static boolean PinAllInstructions = ____; - public static boolean TestPatching = ____; public static boolean FatalUnimplemented = ____; public static int MethodEndBreakpointGuards = 0; public static boolean ZapStackOnMethodEntry = ____; diff -r 62306ff0ccc2 -r ccab1d5f345e graal/GraalCompiler/src/com/sun/c1x/graph/GraphBuilder.java --- a/graal/GraalCompiler/src/com/sun/c1x/graph/GraphBuilder.java Wed May 11 14:42:19 2011 +0200 +++ b/graal/GraalCompiler/src/com/sun/c1x/graph/GraphBuilder.java Wed May 11 14:43:20 2011 +0200 @@ -331,7 +331,7 @@ if (con instanceof RiType) { // this is a load of class constant which might be unresolved RiType riType = (RiType) con; - if (!riType.isResolved() || C1XOptions.TestPatching) { + if (!riType.isResolved()) { ResolveClass rc = new ResolveClass(riType, RiType.Representation.JavaClass, graph, frameState.create(bci())); frameState.push(CiKind.Object, append(rc)); } else { @@ -540,7 +540,7 @@ private void genCheckCast() { int cpi = stream().readCPI(); RiType type = constantPool().lookupType(cpi, CHECKCAST); - boolean isInitialized = !C1XOptions.TestPatching && type.isResolved() && type.isInitialized(); + boolean isInitialized = type.isResolved() && type.isInitialized(); Value typeInstruction = genResolveClass(RiType.Representation.ObjectHub, type, isInitialized, cpi); CheckCast c = new CheckCast(type, typeInstruction, frameState.apop(), graph); frameState.apush(append(c)); @@ -549,7 +549,7 @@ private void genInstanceOf() { int cpi = stream().readCPI(); RiType type = constantPool().lookupType(cpi, INSTANCEOF); - boolean isInitialized = !C1XOptions.TestPatching && type.isResolved() && type.isInitialized(); + boolean isInitialized = type.isResolved() && type.isInitialized(); Value typeInstruction = genResolveClass(RiType.Representation.ObjectHub, type, isInitialized, cpi); InstanceOf i = new InstanceOf(type, typeInstruction, frameState.apop(), graph); frameState.ipush(append(i)); @@ -625,7 +625,7 @@ private void genGetStatic(int cpi, RiField field) { RiType holder = field.holder(); - boolean isInitialized = !C1XOptions.TestPatching && field.isResolved(); + boolean isInitialized = field.isResolved(); CiConstant constantValue = null; FrameState stateBefore = null; if (isInitialized) { @@ -679,7 +679,7 @@ private void genInvokeStatic(RiMethod target, int cpi, RiConstantPool constantPool) { RiType holder = target.holder(); - boolean isInitialized = !C1XOptions.TestPatching && target.isResolved() && holder.isInitialized(); + boolean isInitialized = target.isResolved() && holder.isInitialized(); if (!isInitialized && C1XOptions.ResolveClassBeforeStaticInvoke) { // Re-use the same resolution code as for accessing a static field. Even though // the result of resolution is not used by the invocation (only the side effect