Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(396)

Side by Side Diff: src/x64/macro-assembler-x64.cc

Issue 7014033: Support conversion of clamped double values for pixel arrays in Crankshaft. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: review feedback Created 9 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « src/x64/macro-assembler-x64.h ('k') | test/mjsunit/external-array.js » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2011 the V8 project authors. All rights reserved. 1 // Copyright 2011 the V8 project authors. All rights reserved.
2 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
(...skipping 2552 matching lines...) Expand 10 before | Expand all | Expand 10 after
2563 Label* fail, 2563 Label* fail,
2564 bool is_heap_object) { 2564 bool is_heap_object) {
2565 if (!is_heap_object) { 2565 if (!is_heap_object) {
2566 JumpIfSmi(obj, fail); 2566 JumpIfSmi(obj, fail);
2567 } 2567 }
2568 Cmp(FieldOperand(obj, HeapObject::kMapOffset), map); 2568 Cmp(FieldOperand(obj, HeapObject::kMapOffset), map);
2569 j(not_equal, fail); 2569 j(not_equal, fail);
2570 } 2570 }
2571 2571
2572 2572
2573 void MacroAssembler::ClampUint8(Register reg) {
2574 Label done;
2575 testl(reg, Immediate(0xFFFFFF00));
2576 j(zero, &done, Label::kNear);
2577 setcc(negative, reg); // 1 if negative, 0 if positive.
2578 decb(reg); // 0 if negative, 255 if positive.
2579 bind(&done);
2580 }
2581
2582
2583 void MacroAssembler::ClampDoubleToUint8(XMMRegister input_reg,
2584 XMMRegister temp_xmm_reg,
2585 Register result_reg,
2586 Register temp_reg) {
2587 Label done;
2588 Set(result_reg, 0);
2589 xorps(temp_xmm_reg, temp_xmm_reg);
2590 ucomisd(input_reg, temp_xmm_reg);
2591 j(below, &done, Label::kNear);
2592 uint64_t one_half = BitCast<uint64_t, double>(0.5);
2593 Set(temp_reg, one_half);
2594 movq(temp_xmm_reg, temp_reg);
2595 addsd(temp_xmm_reg, input_reg);
2596 cvttsd2si(result_reg, temp_xmm_reg);
2597 testl(result_reg, Immediate(0xFFFFFF00));
2598 j(zero, &done, Label::kNear);
2599 Set(result_reg, 255);
2600 bind(&done);
2601 }
2602
2603
2573 void MacroAssembler::AbortIfNotNumber(Register object) { 2604 void MacroAssembler::AbortIfNotNumber(Register object) {
2574 Label ok; 2605 Label ok;
2575 Condition is_smi = CheckSmi(object); 2606 Condition is_smi = CheckSmi(object);
2576 j(is_smi, &ok, Label::kNear); 2607 j(is_smi, &ok, Label::kNear);
2577 Cmp(FieldOperand(object, HeapObject::kMapOffset), 2608 Cmp(FieldOperand(object, HeapObject::kMapOffset),
2578 isolate()->factory()->heap_number_map()); 2609 isolate()->factory()->heap_number_map());
2579 Assert(equal, "Operand not a number"); 2610 Assert(equal, "Operand not a number");
2580 bind(&ok); 2611 bind(&ok);
2581 } 2612 }
2582 2613
(...skipping 1070 matching lines...) Expand 10 before | Expand all | Expand 10 after
3653 CPU::FlushICache(address_, size_); 3684 CPU::FlushICache(address_, size_);
3654 3685
3655 // Check that the code was patched as expected. 3686 // Check that the code was patched as expected.
3656 ASSERT(masm_.pc_ == address_ + size_); 3687 ASSERT(masm_.pc_ == address_ + size_);
3657 ASSERT(masm_.reloc_info_writer.pos() == address_ + size_ + Assembler::kGap); 3688 ASSERT(masm_.reloc_info_writer.pos() == address_ + size_ + Assembler::kGap);
3658 } 3689 }
3659 3690
3660 } } // namespace v8::internal 3691 } } // namespace v8::internal
3661 3692
3662 #endif // V8_TARGET_ARCH_X64 3693 #endif // V8_TARGET_ARCH_X64
OLDNEW
« no previous file with comments | « src/x64/macro-assembler-x64.h ('k') | test/mjsunit/external-array.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698