Commit 65fb2cf9 authored by lrn@chromium.org's avatar lrn@chromium.org

Made test.py not mention the defunct --crankshaft flag.

Cleanup of X64 assembler.

Review URL: http://codereview.chromium.org/6764015

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@7477 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
parent 56b53dd5
This diff is collapsed.
...@@ -1589,8 +1589,6 @@ class Assembler : public AssemblerBase { ...@@ -1589,8 +1589,6 @@ class Assembler : public AssemblerBase {
RelocInfoWriter reloc_info_writer; RelocInfoWriter reloc_info_writer;
List< Handle<Code> > code_targets_; List< Handle<Code> > code_targets_;
// push-pop elimination
byte* last_pc_;
PositionsRecorder positions_recorder_; PositionsRecorder positions_recorder_;
......
...@@ -2961,8 +2961,6 @@ void NumberToStringStub::GenerateLookupNumberStringCache(MacroAssembler* masm, ...@@ -2961,8 +2961,6 @@ void NumberToStringStub::GenerateLookupNumberStringCache(MacroAssembler* masm,
times_1, times_1,
FixedArray::kHeaderSize)); FixedArray::kHeaderSize));
__ JumpIfSmi(probe, not_found); __ JumpIfSmi(probe, not_found);
ASSERT(CpuFeatures::IsSupported(SSE2));
CpuFeatures::Scope fscope(SSE2);
__ movsd(xmm0, FieldOperand(object, HeapNumber::kValueOffset)); __ movsd(xmm0, FieldOperand(object, HeapNumber::kValueOffset));
__ movsd(xmm1, FieldOperand(probe, HeapNumber::kValueOffset)); __ movsd(xmm1, FieldOperand(probe, HeapNumber::kValueOffset));
__ ucomisd(xmm0, xmm1); __ ucomisd(xmm0, xmm1);
......
...@@ -600,7 +600,6 @@ void Deoptimizer::DoComputeFrame(TranslationIterator* iterator, ...@@ -600,7 +600,6 @@ void Deoptimizer::DoComputeFrame(TranslationIterator* iterator,
void Deoptimizer::EntryGenerator::Generate() { void Deoptimizer::EntryGenerator::Generate() {
GeneratePrologue(); GeneratePrologue();
CpuFeatures::Scope scope(SSE2);
// Save all general purpose registers before messing with them. // Save all general purpose registers before messing with them.
const int kNumberOfRegisters = Register::kNumRegisters; const int kNumberOfRegisters = Register::kNumRegisters;
......
...@@ -2744,7 +2744,6 @@ void LCodeGen::DoPower(LPower* instr) { ...@@ -2744,7 +2744,6 @@ void LCodeGen::DoPower(LPower* instr) {
ExternalReference::power_double_int_function(isolate()), 2); ExternalReference::power_double_int_function(isolate()), 2);
} else { } else {
ASSERT(exponent_type.IsTagged()); ASSERT(exponent_type.IsTagged());
CpuFeatures::Scope scope(SSE2);
Register right_reg = ToRegister(right); Register right_reg = ToRegister(right);
Label non_smi, call; Label non_smi, call;
......
...@@ -1613,10 +1613,8 @@ LInstruction* LChunkBuilder::DoChange(HChange* instr) { ...@@ -1613,10 +1613,8 @@ LInstruction* LChunkBuilder::DoChange(HChange* instr) {
LOperand* value = UseRegister(instr->value()); LOperand* value = UseRegister(instr->value());
bool needs_check = !instr->value()->type().IsSmi(); bool needs_check = !instr->value()->type().IsSmi();
if (needs_check) { if (needs_check) {
LOperand* xmm_temp = LOperand* xmm_temp = instr->CanTruncateToInt32() ? NULL
(instr->CanTruncateToInt32() && CpuFeatures::IsSupported(SSE3)) : FixedTemp(xmm1);
? NULL
: FixedTemp(xmm1);
LTaggedToI* res = new LTaggedToI(value, xmm_temp); LTaggedToI* res = new LTaggedToI(value, xmm_temp);
return AssignEnvironment(DefineSameAsFirst(res)); return AssignEnvironment(DefineSameAsFirst(res));
} else { } else {
......
...@@ -2199,7 +2199,6 @@ void MacroAssembler::EnterExitFrameEpilogue(int arg_stack_space, ...@@ -2199,7 +2199,6 @@ void MacroAssembler::EnterExitFrameEpilogue(int arg_stack_space,
#endif #endif
// Optionally save all XMM registers. // Optionally save all XMM registers.
if (save_doubles) { if (save_doubles) {
CpuFeatures::Scope scope(SSE2);
int space = XMMRegister::kNumRegisters * kDoubleSize + int space = XMMRegister::kNumRegisters * kDoubleSize +
arg_stack_space * kPointerSize; arg_stack_space * kPointerSize;
subq(rsp, Immediate(space)); subq(rsp, Immediate(space));
......
...@@ -1202,9 +1202,6 @@ def BuildOptions(): ...@@ -1202,9 +1202,6 @@ def BuildOptions():
result.add_option("--nostress", result.add_option("--nostress",
help="Don't run crankshaft --always-opt --stress-op test", help="Don't run crankshaft --always-opt --stress-op test",
default=False, action="store_true") default=False, action="store_true")
result.add_option("--crankshaft",
help="Run with the --crankshaft flag",
default=False, action="store_true")
result.add_option("--shard-count", result.add_option("--shard-count",
help="Split testsuites into this number of shards", help="Split testsuites into this number of shards",
default=1, type="int") default=1, type="int")
...@@ -1245,13 +1242,6 @@ def ProcessOptions(options): ...@@ -1245,13 +1242,6 @@ def ProcessOptions(options):
global VARIANT_FLAGS global VARIANT_FLAGS
if options.stress_only: if options.stress_only:
VARIANT_FLAGS = [['--stress-opt', '--always-opt']] VARIANT_FLAGS = [['--stress-opt', '--always-opt']]
if options.nostress:
VARIANT_FLAGS = [[],['--nocrankshaft']]
if options.crankshaft:
if options.special_command:
options.special_command += " --crankshaft"
else:
options.special_command = "@--crankshaft"
if options.noprof: if options.noprof:
options.scons_flags.append("prof=off") options.scons_flags.append("prof=off")
options.scons_flags.append("profilingsupport=off") options.scons_flags.append("profilingsupport=off")
...@@ -1416,8 +1406,7 @@ def Main(): ...@@ -1416,8 +1406,7 @@ def Main():
'mode': mode, 'mode': mode,
'system': utils.GuessOS(), 'system': utils.GuessOS(),
'arch': options.arch, 'arch': options.arch,
'simulator': options.simulator, 'simulator': options.simulator
'crankshaft': options.crankshaft
} }
test_list = root.ListTests([], path, context, mode, []) test_list = root.ListTests([], path, context, mode, [])
unclassified_tests += test_list unclassified_tests += test_list
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment