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

Side by Side Diff: src/ia32/lithium-codegen-ia32.cc

Issue 19188002: Fix --deopt-every-n-times flag (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 7 years, 5 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/arm/lithium-codegen-arm.cc ('k') | src/mips/lithium-codegen-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 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 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 931 matching lines...) Expand 10 before | Expand all | Expand 10 after
942 FieldOperand(ebx, SharedFunctionInfo::kStressDeoptCounterOffset)); 942 FieldOperand(ebx, SharedFunctionInfo::kStressDeoptCounterOffset));
943 __ sub(Operand(eax), Immediate(Smi::FromInt(1))); 943 __ sub(Operand(eax), Immediate(Smi::FromInt(1)));
944 __ j(not_zero, &no_deopt, Label::kNear); 944 __ j(not_zero, &no_deopt, Label::kNear);
945 if (FLAG_trap_on_deopt) __ int3(); 945 if (FLAG_trap_on_deopt) __ int3();
946 __ mov(eax, Immediate(Smi::FromInt(FLAG_deopt_every_n_times))); 946 __ mov(eax, Immediate(Smi::FromInt(FLAG_deopt_every_n_times)));
947 __ mov(FieldOperand(ebx, SharedFunctionInfo::kStressDeoptCounterOffset), 947 __ mov(FieldOperand(ebx, SharedFunctionInfo::kStressDeoptCounterOffset),
948 eax); 948 eax);
949 __ pop(ebx); 949 __ pop(ebx);
950 __ pop(eax); 950 __ pop(eax);
951 __ popfd(); 951 __ popfd();
952 __ jmp(entry, RelocInfo::RUNTIME_ENTRY); 952 ASSERT(frame_is_built_);
titzer 2013/07/15 13:08:22 Ah ha, missed this case. What about x64?
953 __ call(entry, RelocInfo::RUNTIME_ENTRY);
953 954
954 __ bind(&no_deopt); 955 __ bind(&no_deopt);
955 __ mov(FieldOperand(ebx, SharedFunctionInfo::kStressDeoptCounterOffset), 956 __ mov(FieldOperand(ebx, SharedFunctionInfo::kStressDeoptCounterOffset),
956 eax); 957 eax);
957 __ pop(ebx); 958 __ pop(ebx);
958 __ pop(eax); 959 __ pop(eax);
959 __ popfd(); 960 __ popfd();
960 } 961 }
961 962
962 // Before Instructions which can deopt, we normally flush the x87 stack. But 963 // Before Instructions which can deopt, we normally flush the x87 stack. But
(...skipping 5608 matching lines...) Expand 10 before | Expand all | Expand 10 after
6571 FixedArray::kHeaderSize - kPointerSize)); 6572 FixedArray::kHeaderSize - kPointerSize));
6572 __ bind(&done); 6573 __ bind(&done);
6573 } 6574 }
6574 6575
6575 6576
6576 #undef __ 6577 #undef __
6577 6578
6578 } } // namespace v8::internal 6579 } } // namespace v8::internal
6579 6580
6580 #endif // V8_TARGET_ARCH_IA32 6581 #endif // V8_TARGET_ARCH_IA32
OLDNEW
« no previous file with comments | « src/arm/lithium-codegen-arm.cc ('k') | src/mips/lithium-codegen-mips.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698