Changeset 183162 in webkit for trunk/Source/JavaScriptCore/dfg/DFGCSEPhase.cpp

Timestamp:

Apr 22, 2015, 7:36:08 PM (10 years ago)

Author:

[email protected]

Message:

Nodes should have an optional epoch field
​https://bugs.webkit.org/show_bug.cgi?id=144084

Reviewed by Ryosuke Niwa and Mark Lam.

This makes it easier to do epoch-based analyses on nodes. I plan to do just that in
​https://bugs.webkit.org/show_bug.cgi?id=143735. Currently the epoch field is not yet
used.

• dfg/DFGCPSRethreadingPhase.cpp:

(JSC::DFG::CPSRethreadingPhase::canonicalizeGetLocalFor):

• dfg/DFGCSEPhase.cpp:
• dfg/DFGEpoch.h:

(JSC::DFG::Epoch::fromUnsigned):
(JSC::DFG::Epoch::toUnsigned):

• dfg/DFGGraph.cpp:

(JSC::DFG::Graph::clearReplacements):
(JSC::DFG::Graph::clearEpochs):

• dfg/DFGGraph.h:

(JSC::DFG::Graph::performSubstitutionForEdge):

• dfg/DFGNode.h:

(JSC::DFG::Node::Node):
(JSC::DFG::Node::replaceWith):
(JSC::DFG::Node::replacement):
(JSC::DFG::Node::setReplacement):
(JSC::DFG::Node::epoch):
(JSC::DFG::Node::setEpoch):

• dfg/DFGSSAConversionPhase.cpp:

(JSC::DFG::SSAConversionPhase::run):

File:

• trunk/Source/JavaScriptCore/dfg/DFGCSEPhase.cpp (modified) (2 diffs)

trunk/Source/JavaScriptCore/dfg/DFGCSEPhase.cpp

@@ -236 +236 @@
                 if (m_node->op() == Identity) {
                     m_node->convertToCheck();
-                    m_node->replacement = m_node->child1().node();
+                    m_node->setReplacement(m_node->child1().node());
                     m_changed = true;
                 } else {
@@ -439 +439 @@
                 if (m_node->op() == Identity) {
                     m_node->convertToCheck();
-                    m_node->replacement = m_node->child1().node();
+                    m_node->setReplacement(m_node->child1().node());
                     m_changed = true;
                 } else