# HG changeset patch # User Josef Eisl # Date 1423484959 -3600 # Node ID b689119b6f026fb57df67e3bdf244f384524cd79 # Parent 8d5c61b5cf674afc239107bdde83ed39a5d2466e Remove Backend.shouldAllocateRegisters() (customize Backend.getLowLevelCompilerConfiguration() instead). diff -r 8d5c61b5cf67 -r b689119b6f02 graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/GraalCompiler.java --- a/graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/GraalCompiler.java Mon Feb 09 13:24:53 2015 +0100 +++ b/graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/GraalCompiler.java Mon Feb 09 13:29:19 2015 +0100 @@ -358,9 +358,7 @@ try (Scope s0 = Debug.scope("LowLevelMidTier")) { LowLevelMidTierPhase.Context c = new LowLevelMidTierPhase.Context(); - if (backend.shouldAllocateRegisters()) { - new LinearScanPhase().apply(target, lirGenRes, codeEmittingOrder, linearScanOrder, c); - } + new LinearScanPhase().apply(target, lirGenRes, codeEmittingOrder, linearScanOrder, c); // build frame map if (LSStackSlotAllocator.Options.LSStackSlotAllocation.getValue()) { @@ -368,10 +366,8 @@ } else { new SimpleStackSlotAllocator().apply(target, lirGenRes, codeEmittingOrder, linearScanOrder, c); } - if (backend.shouldAllocateRegisters()) { - // currently we mark locations only if we do register allocation - new LocationMarker().apply(target, lirGenRes, codeEmittingOrder, linearScanOrder, c); - } + // currently we mark locations only if we do register allocation + new LocationMarker().apply(target, lirGenRes, codeEmittingOrder, linearScanOrder, c); } try (Scope s = Debug.scope("LowLevelLowTier")) { diff -r 8d5c61b5cf67 -r b689119b6f02 graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/target/Backend.java --- a/graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/target/Backend.java Mon Feb 09 13:24:53 2015 +0100 +++ b/graal/com.oracle.graal.compiler/src/com/oracle/graal/compiler/target/Backend.java Mon Feb 09 13:29:19 2015 +0100 @@ -104,8 +104,6 @@ public abstract CompilationResultBuilder newCompilationResultBuilder(LIRGenerationResult lirGenResult, FrameMap frameMap, CompilationResult compilationResult, CompilationResultBuilderFactory factory); - public abstract boolean shouldAllocateRegisters(); - public abstract StackIntrospection getStackIntrospection(); /** diff -r 8d5c61b5cf67 -r b689119b6f02 graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotBackend.java --- a/graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotBackend.java Mon Feb 09 13:24:53 2015 +0100 +++ b/graal/com.oracle.graal.hotspot.amd64/src/com/oracle/graal/hotspot/amd64/AMD64HotSpotBackend.java Mon Feb 09 13:29:19 2015 +0100 @@ -63,11 +63,6 @@ } @Override - public boolean shouldAllocateRegisters() { - return true; - } - - @Override public FrameMapBuilder newFrameMapBuilder(RegisterConfig registerConfig) { RegisterConfig registerConfigNonNull = registerConfig == null ? getCodeCache().getRegisterConfig() : registerConfig; return new AMD64FrameMapBuilder(newFrameMap(registerConfigNonNull), getCodeCache(), registerConfigNonNull); diff -r 8d5c61b5cf67 -r b689119b6f02 graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotBackend.java --- a/graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotBackend.java Mon Feb 09 13:24:53 2015 +0100 +++ b/graal/com.oracle.graal.hotspot.sparc/src/com/oracle/graal/hotspot/sparc/SPARCHotSpotBackend.java Mon Feb 09 13:29:19 2015 +0100 @@ -69,11 +69,6 @@ } @Override - public boolean shouldAllocateRegisters() { - return true; - } - - @Override public FrameMapBuilder newFrameMapBuilder(RegisterConfig registerConfig) { RegisterConfig registerConfigNonNull = registerConfig == null ? getCodeCache().getRegisterConfig() : registerConfig; return new SPARCFrameMapBuilder(newFrameMap(registerConfigNonNull), getCodeCache(), registerConfigNonNull);