Changeset 153280 in webkit for trunk/Source/JavaScriptCore/dfg/DFGAbstractState.h

Timestamp:

Jul 24, 2013, 9:04:57 PM (12 years ago)

Author:

[email protected]

Message:

fourthTier: CFA should consider live-at-head for clobbering and dumping
​https://bugs.webkit.org/show_bug.cgi?id=118857

Reviewed by Mark Hahnenberg.

• clobberStructures() was not considering nodes live-at-head when in SSA form. This means it would fail to clobber some structures.

• dump() was not considering nodes live-at-head when in SSA form. This means it wouldn't dump everything that you might be interested in.

• AbstractState::m_currentNode is a useless variable and we should get rid of it.

• dfg/DFGAbstractState.cpp:

(JSC::DFG::AbstractState::AbstractState):
(JSC::DFG::AbstractState::beginBasicBlock):
(JSC::DFG::AbstractState::reset):
(JSC::DFG::AbstractState::startExecuting):
(JSC::DFG::AbstractState::clobberStructures):
(JSC::DFG::AbstractState::dump):

• dfg/DFGAbstractState.h:

(AbstractState):

File:

• trunk/Source/JavaScriptCore/dfg/DFGAbstractState.h (modified) (1 diff)

trunk/Source/JavaScriptCore/dfg/DFGAbstractState.h

@@ -314 +314 @@
     Operands<AbstractValue> m_variables;
     BasicBlock* m_block;
-    Node* m_currentNode;
     
     bool m_haveStructures;