diff options
| author | Filip Pizlo <fpizlo@apple.com> | 2013-03-21 15:50:35 +0100 |
|---|---|---|
| committer | The Qt Project <gerrit-noreply@qt-project.org> | 2013-03-27 11:57:36 +0100 |
| commit | 2a43c884303d7a002f6e164aaa5bb7b301425563 (patch) | |
| tree | b8758210a734eddf073752fbfd440b0306f7db0d | |
| parent | 09961e4b798b98e1e35688ce692094186a8f5d07 (diff) | |
DFG Phantom node should be honest about the fact that it can exitv5.1.0-alpha1
https://bugs.webkit.org/show_bug.cgi?id=111115
Reviewed by Mark Hahnenberg.
The chances of this having cause serious issues are low, since most clients of the
NodeDoesNotExit flag run after CFA and CFA updates this properly. But one possible
case of badness is if the ByteCodeParser inserted a Phantom with a type check in
between a LogicalNot and a Branch; then that peephole optimization in Fixup might
go slightly wrong.
* dfg/DFGNodeType.h:
(DFG):
Change-Id: I568e22c3577979de0eeea945de519a920ee212b6
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@144373 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Reviewed-by: Jocelyn Turcotte <jocelyn.turcotte@digia.com>
| -rw-r--r-- | Source/JavaScriptCore/dfg/DFGNodeType.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Source/JavaScriptCore/dfg/DFGNodeType.h b/Source/JavaScriptCore/dfg/DFGNodeType.h index cbcdde660..b3fd78785 100644 --- a/Source/JavaScriptCore/dfg/DFGNodeType.h +++ b/Source/JavaScriptCore/dfg/DFGNodeType.h @@ -58,7 +58,7 @@ namespace JSC { namespace DFG { /* VariableAccessData, and thus will share predictions. */\ macro(GetLocal, NodeResultJS) \ macro(SetLocal, 0) \ - macro(Phantom, NodeMustGenerate | NodeDoesNotExit) \ + macro(Phantom, NodeMustGenerate) \ macro(Nop, 0 | NodeDoesNotExit) \ macro(Phi, 0 | NodeDoesNotExit) \ macro(Flush, NodeMustGenerate | NodeDoesNotExit) \ |