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

Side by Side Diff: src/mips/assembler-mips.h

Issue 10990110: MIPS: Changed "marked" nops to use sll(zero_reg, at, type). (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 8 years, 2 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 | « no previous file | src/mips/assembler-mips.cc » ('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 (c) 1994-2006 Sun Microsystems Inc. 1 // Copyright (c) 1994-2006 Sun Microsystems Inc.
2 // All Rights Reserved. 2 // All Rights Reserved.
3 // 3 //
4 // Redistribution and use in source and binary forms, with or without 4 // Redistribution and use in source and binary forms, with or without
5 // modification, are permitted provided that the following conditions are 5 // modification, are permitted provided that the following conditions are
6 // met: 6 // met:
7 // 7 //
8 // - Redistributions of source code must retain the above copyright notice, 8 // - Redistributions of source code must retain the above copyright notice,
9 // this list of conditions and the following disclaimer. 9 // this list of conditions and the following disclaimer.
10 // 10 //
(...skipping 645 matching lines...) Expand 10 before | Expand all | Expand 10 after
656 DEBUG_BREAK_NOP, 656 DEBUG_BREAK_NOP,
657 // IC markers. 657 // IC markers.
658 PROPERTY_ACCESS_INLINED, 658 PROPERTY_ACCESS_INLINED,
659 PROPERTY_ACCESS_INLINED_CONTEXT, 659 PROPERTY_ACCESS_INLINED_CONTEXT,
660 PROPERTY_ACCESS_INLINED_CONTEXT_DONT_DELETE, 660 PROPERTY_ACCESS_INLINED_CONTEXT_DONT_DELETE,
661 // Helper values. 661 // Helper values.
662 LAST_CODE_MARKER, 662 LAST_CODE_MARKER,
663 FIRST_IC_MARKER = PROPERTY_ACCESS_INLINED 663 FIRST_IC_MARKER = PROPERTY_ACCESS_INLINED
664 }; 664 };
665 665
666 // Type == 0 is the default non-marking type. 666 // Type == 0 is the default non-marking nop. For mips this is a
667 // sll(zero_reg, zero_reg, 0). We use rt_reg == at for non-zero
668 // marking, to avoid conflict with ssnop and ehb instructions.
667 void nop(unsigned int type = 0) { 669 void nop(unsigned int type = 0) {
668 ASSERT(type < 32); 670 ASSERT(type < 32);
669 sll(zero_reg, zero_reg, type, true); 671 Register nop_rt_reg = (type == 0) ? zero_reg : at;
672 sll(zero_reg, nop_rt_reg, type, true);
670 } 673 }
671 674
672 675
673 // --------Branch-and-jump-instructions---------- 676 // --------Branch-and-jump-instructions----------
674 // We don't use likely variant of instructions. 677 // We don't use likely variant of instructions.
675 void b(int16_t offset); 678 void b(int16_t offset);
676 void b(Label* L) { b(branch_offset(L, false)>>2); } 679 void b(Label* L) { b(branch_offset(L, false)>>2); }
677 void bal(int16_t offset); 680 void bal(int16_t offset);
678 void bal(Label* L) { bal(branch_offset(L, false)>>2); } 681 void bal(Label* L) { bal(branch_offset(L, false)>>2); }
679 682
(...skipping 602 matching lines...) Expand 10 before | Expand all | Expand 10 after
1282 class EnsureSpace BASE_EMBEDDED { 1285 class EnsureSpace BASE_EMBEDDED {
1283 public: 1286 public:
1284 explicit EnsureSpace(Assembler* assembler) { 1287 explicit EnsureSpace(Assembler* assembler) {
1285 assembler->CheckBuffer(); 1288 assembler->CheckBuffer();
1286 } 1289 }
1287 }; 1290 };
1288 1291
1289 } } // namespace v8::internal 1292 } } // namespace v8::internal
1290 1293
1291 #endif // V8_ARM_ASSEMBLER_MIPS_H_ 1294 #endif // V8_ARM_ASSEMBLER_MIPS_H_
OLDNEW
« no previous file with comments | « no previous file | src/mips/assembler-mips.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698