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

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

Issue 113458: First round of size reduction for JumpTargets. Reduce their size by... (Closed) Base URL: http://v8.googlecode.com/svn/branches/bleeding_edge/
Patch Set: '' Created 11 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
OLDNEW
1 // Copyright 2006-2008 the V8 project authors. All rights reserved. 1 // Copyright 2006-2008 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 613 matching lines...) Expand 10 before | Expand all | Expand 10 after
624 j(not_equal, miss, not_taken); 624 j(not_equal, miss, not_taken);
625 625
626 bind(&same_contexts); 626 bind(&same_contexts);
627 } 627 }
628 628
629 629
630 void MacroAssembler::NegativeZeroTest(CodeGenerator* cgen, 630 void MacroAssembler::NegativeZeroTest(CodeGenerator* cgen,
631 Register result, 631 Register result,
632 Register op, 632 Register op,
633 JumpTarget* then_target) { 633 JumpTarget* then_target) {
634 JumpTarget ok(cgen); 634 JumpTarget ok;
635 test(result, Operand(result)); 635 test(result, Operand(result));
636 ok.Branch(not_zero, taken); 636 ok.Branch(not_zero, taken);
637 test(op, Operand(op)); 637 test(op, Operand(op));
638 then_target->Branch(sign, not_taken); 638 then_target->Branch(sign, not_taken);
639 ok.Bind(); 639 ok.Bind();
640 } 640 }
641 641
642 642
643 void MacroAssembler::NegativeZeroTest(Register result, 643 void MacroAssembler::NegativeZeroTest(Register result,
644 Register op, 644 Register op,
(...skipping 397 matching lines...) Expand 10 before | Expand all | Expand 10 after
1042 // Indicate that code has changed. 1042 // Indicate that code has changed.
1043 CPU::FlushICache(address_, size_); 1043 CPU::FlushICache(address_, size_);
1044 1044
1045 // Check that the code was patched as expected. 1045 // Check that the code was patched as expected.
1046 ASSERT(masm_.pc_ == address_ + size_); 1046 ASSERT(masm_.pc_ == address_ + size_);
1047 ASSERT(masm_.reloc_info_writer.pos() == address_ + size_ + Assembler::kGap); 1047 ASSERT(masm_.reloc_info_writer.pos() == address_ + size_ + Assembler::kGap);
1048 } 1048 }
1049 1049
1050 1050
1051 } } // namespace v8::internal 1051 } } // namespace v8::internal
OLDNEW
« no previous file with comments | « src/ia32/jump-target-ia32.cc ('k') | src/jump-target.h » ('j') | src/jump-target.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698