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

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

Issue 2855018: X64: Change strategy for spilling to match ia32. It's just better. (Closed)
Patch Set: Addressed review comments Created 10 years, 6 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
« no previous file with comments | « src/ia32/assembler-ia32.cc ('k') | src/x64/assembler-x64.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 481 matching lines...) Expand 10 before | Expand all | Expand 10 after
492 // - Instructions on 32-bit (doubleword) operands/registers use 'l'. 492 // - Instructions on 32-bit (doubleword) operands/registers use 'l'.
493 // - Instructions on 64-bit (quadword) operands/registers use 'q'. 493 // - Instructions on 64-bit (quadword) operands/registers use 'q'.
494 // 494 //
495 // Some mnemonics, such as "and", are the same as C++ keywords. 495 // Some mnemonics, such as "and", are the same as C++ keywords.
496 // Naming conflicts with C++ keywords are resolved by adding a trailing '_'. 496 // Naming conflicts with C++ keywords are resolved by adding a trailing '_'.
497 497
498 // Insert the smallest number of nop instructions 498 // Insert the smallest number of nop instructions
499 // possible to align the pc offset to a multiple 499 // possible to align the pc offset to a multiple
500 // of m. m must be a power of 2. 500 // of m. m must be a power of 2.
501 void Align(int m); 501 void Align(int m);
502 // Aligns code to something that's optimal for a jump target for the platform.
503 void CodeTargetAlign();
502 504
503 // Stack 505 // Stack
504 void pushfq(); 506 void pushfq();
505 void popfq(); 507 void popfq();
506 508
507 void push(Immediate value); 509 void push(Immediate value);
508 void push(Register src); 510 void push(Register src);
509 void push(const Operand& src); 511 void push(const Operand& src);
510 void push(Label* label, RelocInfo::Mode relocation_mode); 512 void push(Label* label, RelocInfo::Mode relocation_mode);
511 513
(...skipping 904 matching lines...) Expand 10 before | Expand all | Expand 10 after
1416 private: 1418 private:
1417 Assembler* assembler_; 1419 Assembler* assembler_;
1418 #ifdef DEBUG 1420 #ifdef DEBUG
1419 int space_before_; 1421 int space_before_;
1420 #endif 1422 #endif
1421 }; 1423 };
1422 1424
1423 } } // namespace v8::internal 1425 } } // namespace v8::internal
1424 1426
1425 #endif // V8_X64_ASSEMBLER_X64_H_ 1427 #endif // V8_X64_ASSEMBLER_X64_H_
OLDNEW
« no previous file with comments | « src/ia32/assembler-ia32.cc ('k') | src/x64/assembler-x64.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698