# HG changeset patch # User Christian Humer # Date 1363201482 -3600 # Node ID 4dc7034317ecae41de196708ed76affa1e3ba4d6 # Parent 1384c39e2c53f925ceacd83fb689ed45ea97b220 Cleanup. diff -r 1384c39e2c53 -r 4dc7034317ec graal/com.oracle.truffle.codegen.processor/src/com/oracle/truffle/codegen/processor/node/GenericParser.java --- a/graal/com.oracle.truffle.codegen.processor/src/com/oracle/truffle/codegen/processor/node/GenericParser.java Wed Mar 13 20:00:33 2013 +0100 +++ b/graal/com.oracle.truffle.codegen.processor/src/com/oracle/truffle/codegen/processor/node/GenericParser.java Wed Mar 13 20:04:42 2013 +0100 @@ -61,7 +61,7 @@ @Override public SpecializationData create(TemplateMethod method) { - SpecializationData data = new SpecializationData(method, true, false, false); + SpecializationData data = new SpecializationData(method, true, false); data.setUseSpecializationsForGeneric(Utils.getAnnotationValueBoolean(data.getMarkerAnnotation(), "useSpecializations")); return data; } diff -r 1384c39e2c53 -r 4dc7034317ec graal/com.oracle.truffle.codegen.processor/src/com/oracle/truffle/codegen/processor/node/NodeCodeGenerator.java --- a/graal/com.oracle.truffle.codegen.processor/src/com/oracle/truffle/codegen/processor/node/NodeCodeGenerator.java Wed Mar 13 20:00:33 2013 +0100 +++ b/graal/com.oracle.truffle.codegen.processor/src/com/oracle/truffle/codegen/processor/node/NodeCodeGenerator.java Wed Mar 13 20:04:42 2013 +0100 @@ -1001,7 +1001,7 @@ builder.startTryBlock(); } - if (specialization.isSynthetic()) { + if (specialization.getMethod() == null) { emitEncounteredSynthetic(builder); } else { builder.startReturn(); diff -r 1384c39e2c53 -r 4dc7034317ec graal/com.oracle.truffle.codegen.processor/src/com/oracle/truffle/codegen/processor/node/NodeParser.java --- a/graal/com.oracle.truffle.codegen.processor/src/com/oracle/truffle/codegen/processor/node/NodeParser.java Wed Mar 13 20:00:33 2013 +0100 +++ b/graal/com.oracle.truffle.codegen.processor/src/com/oracle/truffle/codegen/processor/node/NodeParser.java Wed Mar 13 20:04:42 2013 +0100 @@ -290,10 +290,7 @@ MethodSpec specification = parser.createDefaultMethodSpec(specialization.getMethod(), null, null); ExecutableTypeData anyGenericReturnType = node.findAnyGenericExecutableType(context); - if (anyGenericReturnType == null) { - // TODO fail invalid executable type. should be validated by field. (assertion -// failure!?) - } + assert anyGenericReturnType != null; ActualParameter returnType = new ActualParameter(specification.getReturnType(), anyGenericReturnType.getType().getPrimitiveType(), 0, false); List parameters = new ArrayList<>(); @@ -305,15 +302,13 @@ actualType = specializationParameter.getActualType(); } else { ExecutableTypeData paramType = field.getNodeData().findAnyGenericExecutableType(context); - if (paramType == null) { - // TODO fail - } + assert paramType != null; actualType = paramType.getType().getPrimitiveType(); } parameters.add(new ActualParameter(parameterSpec, actualType, specializationParameter.getIndex(), specializationParameter.isHidden())); } TemplateMethod genericMethod = new TemplateMethod("Generic", node, specification, null, null, returnType, parameters); - genericSpecialization = new SpecializationData(genericMethod, true, false, true); + genericSpecialization = new SpecializationData(genericMethod, true, false); specializations.add(genericSpecialization); } @@ -322,7 +317,7 @@ CodeExecutableElement uninitializedMethod = new CodeExecutableElement(Utils.modifiers(Modifier.PUBLIC), context.getType(void.class), "doUninitialized"); TemplateMethod uninializedMethod = new TemplateMethod("Uninitialized", node, genericSpecialization.getSpecification(), uninitializedMethod, genericSpecialization.getMarkerAnnotation(), genericSpecialization.getReturnType(), genericSpecialization.getParameters()); - specializations.add(new SpecializationData(uninializedMethod, false, true, true)); + specializations.add(new SpecializationData(uninializedMethod, false, true)); } Collections.sort(specializations, new Comparator() { diff -r 1384c39e2c53 -r 4dc7034317ec graal/com.oracle.truffle.codegen.processor/src/com/oracle/truffle/codegen/processor/node/SpecializationData.java --- a/graal/com.oracle.truffle.codegen.processor/src/com/oracle/truffle/codegen/processor/node/SpecializationData.java Wed Mar 13 20:00:33 2013 +0100 +++ b/graal/com.oracle.truffle.codegen.processor/src/com/oracle/truffle/codegen/processor/node/SpecializationData.java Wed Mar 13 20:04:42 2013 +0100 @@ -39,14 +39,11 @@ private boolean useSpecializationsForGeneric = true; private NodeData node; - private final boolean synthetic; - public SpecializationData(TemplateMethod template, int order, List exceptions) { super(template); this.order = order; this.generic = false; this.uninitialized = false; - this.synthetic = false; this.exceptions = exceptions; for (SpecializationThrowsData exception : exceptions) { @@ -54,14 +51,13 @@ } } - public SpecializationData(TemplateMethod template, boolean generic, boolean uninitialized, boolean synthetic) { + public SpecializationData(TemplateMethod template, boolean generic, boolean uninitialized) { super(template); this.order = Specialization.DEFAULT_ORDER; this.generic = generic; this.uninitialized = uninitialized; this.exceptions = Collections.emptyList(); this.guards = new ArrayList<>(); - this.synthetic = synthetic; } @Override @@ -108,10 +104,6 @@ this.guards = guards; } - public boolean isSynthetic() { - return synthetic; - } - public int getOrder() { return order; } diff -r 1384c39e2c53 -r 4dc7034317ec graal/com.oracle.truffle.codegen.processor/src/com/oracle/truffle/codegen/processor/template/MessageContainer.java --- a/graal/com.oracle.truffle.codegen.processor/src/com/oracle/truffle/codegen/processor/template/MessageContainer.java Wed Mar 13 20:00:33 2013 +0100 +++ b/graal/com.oracle.truffle.codegen.processor/src/com/oracle/truffle/codegen/processor/template/MessageContainer.java Wed Mar 13 20:04:42 2013 +0100 @@ -146,7 +146,7 @@ return messages; } - public static class Message { + public static final class Message { private final MessageContainer originalContainer; private final String text;