Revision: 21612
Author: [email protected]
Date: Mon Jun 2 13:53:21 2014 UTC
Log: X87: Don't push eax/rax before calling Runtime_Abort
Additionally delete the unused Throw(BailoutReason)
Port r21582 (41dd40b)
BUG=
[email protected]
Review URL: https://codereview.chromium.org/305013012
http://code.google.com/p/v8/source/detail?r=21612
Modified:
/branches/bleeding_edge/src/x87/macro-assembler-x87.cc
/branches/bleeding_edge/src/x87/macro-assembler-x87.h
=======================================
--- /branches/bleeding_edge/src/x87/macro-assembler-x87.cc Wed May 28
08:18:39 2014 UTC
+++ /branches/bleeding_edge/src/x87/macro-assembler-x87.cc Mon Jun 2
13:53:21 2014 UTC
@@ -2683,7 +2683,6 @@
}
#endif
- push(eax);
push(Immediate(reinterpret_cast<intptr_t>(Smi::FromInt(reason))));
// Disable stub call restrictions to always allow calls to abort.
if (!has_frame_) {
@@ -2697,40 +2696,6 @@
// will not return here
int3();
}
-
-
-void MacroAssembler::Throw(BailoutReason reason) {
-#ifdef DEBUG
- const char* msg = GetBailoutReason(reason);
- if (msg != NULL) {
- RecordComment("Throw message: ");
- RecordComment(msg);
- }
-#endif
-
- push(eax);
- push(Immediate(Smi::FromInt(reason)));
- // Disable stub call restrictions to always allow calls to throw.
- if (!has_frame_) {
- // We don't actually want to generate a pile of code for this, so just
- // claim there is a stack frame, without generating one.
- FrameScope scope(this, StackFrame::NONE);
- CallRuntime(Runtime::kHiddenThrowMessage, 1);
- } else {
- CallRuntime(Runtime::kHiddenThrowMessage, 1);
- }
- // will not return here
- int3();
-}
-
-
-void MacroAssembler::ThrowIf(Condition cc, BailoutReason reason) {
- Label L;
- j(NegateCondition(cc), &L);
- Throw(reason);
- // will not return here
- bind(&L);
-}
void MacroAssembler::LoadInstanceDescriptors(Register map,
=======================================
--- /branches/bleeding_edge/src/x87/macro-assembler-x87.h Wed May 28
08:18:39 2014 UTC
+++ /branches/bleeding_edge/src/x87/macro-assembler-x87.h Mon Jun 2
13:53:21 2014 UTC
@@ -532,12 +532,6 @@
// Throw past all JS frames to the top JS entry frame.
void ThrowUncatchable(Register value);
- // Throw a message string as an exception.
- void Throw(BailoutReason reason);
-
- // Throw a message string as an exception if a condition is not true.
- void ThrowIf(Condition cc, BailoutReason reason);
-
//
---------------------------------------------------------------------------
// Inline caching support
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
