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

Side by Side Diff: src/a64/builtins-a64.cc

Issue 133443009: A64: Synchronize with r17441. (Closed) Base URL: https://v8.googlecode.com/svn/branches/experimental/a64
Patch Set: Created 6 years, 10 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/a64/assembler-a64-inl.h ('k') | src/a64/code-stubs-a64.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 2013 the V8 project authors. All rights reserved. 1 // Copyright 2013 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 793 matching lines...) Expand 10 before | Expand all | Expand 10 after
804 GenerateMakeCodeYoungAgainCommon(masm); \ 804 GenerateMakeCodeYoungAgainCommon(masm); \
805 } \ 805 } \
806 void Builtins::Generate_Make##C##CodeYoungAgainOddMarking( \ 806 void Builtins::Generate_Make##C##CodeYoungAgainOddMarking( \
807 MacroAssembler* masm) { \ 807 MacroAssembler* masm) { \
808 GenerateMakeCodeYoungAgainCommon(masm); \ 808 GenerateMakeCodeYoungAgainCommon(masm); \
809 } 809 }
810 CODE_AGE_LIST(DEFINE_CODE_AGE_BUILTIN_GENERATOR) 810 CODE_AGE_LIST(DEFINE_CODE_AGE_BUILTIN_GENERATOR)
811 #undef DEFINE_CODE_AGE_BUILTIN_GENERATOR 811 #undef DEFINE_CODE_AGE_BUILTIN_GENERATOR
812 812
813 813
814 void Builtins::Generate_MarkCodeAsExecutedOnce(MacroAssembler* masm) {
815 // For now, as in GenerateMakeCodeYoungAgainCommon, we are relying on the fact
816 // that make_code_young doesn't do any garbage collection which allows us to
817 // save/restore the registers without worrying about which of them contain
818 // pointers.
819
820 // The following caller-saved registers must be saved and restored when
821 // calling through to the runtime:
822 // x0 - The address from which to resume execution.
823 // x1 - isolate
824 // lr - The return address for the JSFunction itself. It has not yet been
825 // preserved on the stack because the frame setup code was replaced
826 // with a call to this stub, to handle code ageing.
827 {
828 FrameScope scope(masm, StackFrame::MANUAL);
829 __ Push(x0, x1, fp, lr);
830 __ Mov(x1, Operand(ExternalReference::isolate_address(masm->isolate())));
831 __ CallCFunction(
832 ExternalReference::get_make_code_young_function(masm->isolate()), 2);
833 __ Pop(lr, fp, x1, x0);
834
835 // Perform prologue operations usually performed by the young code stub.
836 __ EmitFrameSetupForCodeAgePatching(masm);
837 }
838
839 // Jump to point after the code-age stub.
840 __ Add(x0, x0, kCodeAgeSequenceSize);
841 __ Br(x0);
842 }
843
844
845 void Builtins::Generate_MarkCodeAsExecutedTwice(MacroAssembler* masm) {
846 GenerateMakeCodeYoungAgainCommon(masm);
847 }
848
849
814 void Builtins::Generate_NotifyStubFailure(MacroAssembler* masm) { 850 void Builtins::Generate_NotifyStubFailure(MacroAssembler* masm) {
815 { 851 {
816 FrameScope scope(masm, StackFrame::INTERNAL); 852 FrameScope scope(masm, StackFrame::INTERNAL);
817 853
818 // Preserve registers across notification, this is important for compiled 854 // Preserve registers across notification, this is important for compiled
819 // stubs that tail call the runtime on deopts passing their parameters in 855 // stubs that tail call the runtime on deopts passing their parameters in
820 // registers. 856 // registers.
821 // TODO(jbramley): Is it correct (and appropriate) to use safepoint 857 // TODO(jbramley): Is it correct (and appropriate) to use safepoint
822 // registers here? According to the comment above, we should only need to 858 // registers here? According to the comment above, we should only need to
823 // preserve the registers with parameters. 859 // preserve the registers with parameters.
(...skipping 612 matching lines...) Expand 10 before | Expand all | Expand 10 after
1436 __ Bind(&dont_adapt_arguments); 1472 __ Bind(&dont_adapt_arguments);
1437 __ Jump(x3); 1473 __ Jump(x3);
1438 } 1474 }
1439 1475
1440 1476
1441 #undef __ 1477 #undef __
1442 1478
1443 } } // namespace v8::internal 1479 } } // namespace v8::internal
1444 1480
1445 #endif // V8_TARGET_ARCH_ARM 1481 #endif // V8_TARGET_ARCH_ARM
OLDNEW
« no previous file with comments | « src/a64/assembler-a64-inl.h ('k') | src/a64/code-stubs-a64.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698