Reviewers: Sven Panne,
Message:
Committed patchset #1 manually as r16718.
Description:
revert 16717 for breaking everything
[email protected]
BUG=
Committed: https://code.google.com/p/v8/source/detail?r=16718
Please review this at https://codereview.chromium.org/23708039/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files (+5, -10 lines):
M src/flag-definitions.h
M src/platform-posix.cc
M src/platform-win32.cc
M src/platform.h
Index: src/flag-definitions.h
diff --git a/src/flag-definitions.h b/src/flag-definitions.h
index
8e77ac809e30b72c6d6f6037d62898932e268191..cc153af2542dfca612c4e12c579318931190acbd
100644
--- a/src/flag-definitions.h
+++ b/src/flag-definitions.h
@@ -473,7 +473,7 @@ DEFINE_bool(debugger_auto_break, true,
"automatically set the debug break flag when debugger commands
are "
"in the queue")
DEFINE_bool(enable_liveedit, true, "enable liveedit experimental feature")
-DEFINE_bool(break_on_abort, false, "always cause a debug break before
aborting")
+DEFINE_bool(break_on_abort, true, "always cause a debug break before
aborting")
// execution.cc
// Slightly less than 1MB on 64-bit, since Windows' default stack size for
Index: src/platform-posix.cc
diff --git a/src/platform-posix.cc b/src/platform-posix.cc
index
69303bfd80ce42278c830edfbd919d7391b83cb3..fe27eaf71f437fcc099ffdebb8e24f60c999921f
100644
--- a/src/platform-posix.cc
+++ b/src/platform-posix.cc
@@ -222,7 +222,7 @@ void OS::Abort() {
if (FLAG_break_on_abort) {
DebugBreak();
}
- V8_IMMEDIATE_CRASH();
+ abort();
}
Index: src/platform-win32.cc
diff --git a/src/platform-win32.cc b/src/platform-win32.cc
index
7a90920a88b1809748eef69497c8434f80a0b402..073b21a011aae732b7db5dc15ca3b18c85e3858b
100644
--- a/src/platform-win32.cc
+++ b/src/platform-win32.cc
@@ -872,8 +872,10 @@ void OS::Sleep(int milliseconds) {
void OS::Abort() {
if (IsDebuggerPresent() || FLAG_break_on_abort) {
DebugBreak();
+ } else {
+ // Make the MSVCRT do a silent abort.
+ raise(SIGABRT);
}
- V8_IMMEDIATE_CRASH();
}
Index: src/platform.h
diff --git a/src/platform.h b/src/platform.h
index
13c126bf42312ec8a5708a2ebcfe5fc670531457..aa50cb4c7230f4dcfa9768faaddb0a4ed80e0b30
100644
--- a/src/platform.h
+++ b/src/platform.h
@@ -51,13 +51,6 @@
#include "utils.h"
#include "v8globals.h"
-// This is lined up with blink's method of crashing.
-#if defined(__GNUC__)
-#define V8_IMMEDIATE_CRASH() __builtin_trap()
-#else
-#define V8_IMMEDIATE_CRASH() ((void(*)())0)()
-#endif
-
#ifdef __sun
# ifndef signbit
namespace std {
--
--
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/groups/opt_out.
