Ignore:
Timestamp:
Sep 10, 2013, 3:16:00 PM (12 years ago)
Author:
[email protected]
Message:

We should say Int32 when we mean Int32. Saying Integer is just weird.

Rubber stamped by Mark Hahnenberg.

  • dfg/DFGAbstractInterpreterInlines.h:

(JSC::DFG::::executeEffects):

  • dfg/DFGFixupPhase.cpp:

(JSC::DFG::FixupPhase::fixupNode):
(JSC::DFG::FixupPhase::fixupToPrimitive):
(JSC::DFG::FixupPhase::fixIntEdge):
(JSC::DFG::FixupPhase::truncateConstantsIfNecessary):
(JSC::DFG::FixupPhase::attemptToMakeIntegerAdd):

  • dfg/DFGGraph.h:

(JSC::DFG::Graph::addSpeculationMode):
(JSC::DFG::Graph::valueAddSpeculationMode):
(JSC::DFG::Graph::arithAddSpeculationMode):
(JSC::DFG::Graph::addShouldSpeculateInt32):
(JSC::DFG::Graph::mulShouldSpeculateInt32):
(JSC::DFG::Graph::negateShouldSpeculateInt32):
(JSC::DFG::Graph::addImmediateShouldSpeculateInt32):
(JSC::DFG::Graph::mulImmediateShouldSpeculateInt32):

  • dfg/DFGNode.h:

(JSC::DFG::Node::shouldSpeculateInt32):
(JSC::DFG::Node::shouldSpeculateInt32ForArithmetic):
(JSC::DFG::Node::shouldSpeculateInt32ExpectingDefined):
(JSC::DFG::Node::canSpeculateInt32):

  • dfg/DFGNodeFlags.h:

(JSC::DFG::nodeCanSpeculateInt32):

  • dfg/DFGPredictionPropagationPhase.cpp:

(JSC::DFG::PredictionPropagationPhase::propagate):
(JSC::DFG::PredictionPropagationPhase::doDoubleVoting):

  • dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::arrayify):
(JSC::DFG::GPRTemporary::GPRTemporary):
(JSC::DFG::SpeculativeJIT::compilePeepHoleIntegerBranch):
(JSC::DFG::SpeculativeJIT::compileValueToInt32):
(JSC::DFG::SpeculativeJIT::compileUInt32ToNumber):
(JSC::DFG::SpeculativeJIT::compileInt32ToDouble):
(JSC::DFG::SpeculativeJIT::compileGetByValOnIntTypedArray):
(JSC::DFG::SpeculativeJIT::compilePutByValForIntTypedArray):
(JSC::DFG::SpeculativeJIT::compileAdd):
(JSC::DFG::SpeculativeJIT::compileArithSub):
(JSC::DFG::SpeculativeJIT::compileArithNegate):
(JSC::DFG::SpeculativeJIT::compileArithIMul):
(JSC::DFG::SpeculativeJIT::compileArithMul):
(JSC::DFG::SpeculativeJIT::compileArithDiv):
(JSC::DFG::SpeculativeJIT::compileArithMod):
(JSC::DFG::SpeculativeJIT::compileNewTypedArray):
(JSC::DFG::SpeculativeJIT::speculateInt32):
(JSC::DFG::SpeculativeJIT::emitSwitchImm):

  • dfg/DFGSpeculativeJIT.h:

(JSC::DFG::SpeculateInt32Operand::SpeculateInt32Operand):
(JSC::DFG::SpeculateInt32Operand::~SpeculateInt32Operand):

  • dfg/DFGSpeculativeJIT32_64.cpp:

(JSC::DFG::SpeculativeJIT::compileIntegerCompare):
(JSC::DFG::SpeculativeJIT::compileLogicalNot):
(JSC::DFG::SpeculativeJIT::emitBranch):
(JSC::DFG::SpeculativeJIT::compile):

  • dfg/DFGSpeculativeJIT64.cpp:

(JSC::DFG::SpeculativeJIT::compileIntegerCompare):
(JSC::DFG::SpeculativeJIT::compileLogicalNot):
(JSC::DFG::SpeculativeJIT::emitBranch):
(JSC::DFG::SpeculativeJIT::compile):

  • ftl/FTLLowerDFGToLLVM.cpp:

(JSC::FTL::LowerDFGToLLVM::compileUInt32ToNumber):
(JSC::FTL::LowerDFGToLLVM::compileGetByVal):

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/Source/JavaScriptCore/dfg/DFGPredictionPropagationPhase.cpp

    r155023 r155482  
    198198
    199199        case UInt32ToNumber: {
    200             if (nodeCanSpeculateInteger(node->arithNodeFlags()))
     200            if (nodeCanSpeculateInt32(node->arithNodeFlags()))
    201201                changed |= mergePrediction(SpecInt32);
    202202            else
    … …  
    211211            if (left && right) {
    212212                if (isNumberSpeculationExpectingDefined(left) && isNumberSpeculationExpectingDefined(right)) {
    213                     if (m_graph.addSpeculationMode(node) != DontSpeculateInteger)
     213                    if (m_graph.addSpeculationMode(node) != DontSpeculateInt32)
    214214                        changed |= mergePrediction(SpecInt32);
    215215                    else
    … …  
    229229           
    230230            if (left && right) {
    231                 if (m_graph.addSpeculationMode(node) != DontSpeculateInteger)
     231                if (m_graph.addSpeculationMode(node) != DontSpeculateInt32)
    232232                    changed |= mergePrediction(SpecInt32);
    233233                else
    … …  
    242242           
    243243            if (left && right) {
    244                 if (m_graph.addSpeculationMode(node) != DontSpeculateInteger)
     244                if (m_graph.addSpeculationMode(node) != DontSpeculateInt32)
    245245                    changed |= mergePrediction(SpecInt32);
    246246                else
    … …  
    252252        case ArithNegate:
    253253            if (node->child1()->prediction()) {
    254                 if (m_graph.negateShouldSpeculateInteger(node))
     254                if (m_graph.negateShouldSpeculateInt32(node))
    255255                    changed |= mergePrediction(SpecInt32);
    256256                else
    … …  
    265265           
    266266            if (left && right) {
    267                 if (Node::shouldSpeculateIntegerForArithmetic(node->child1().node(), node->child2().node())
    268                     && nodeCanSpeculateInteger(node->arithNodeFlags()))
     267                if (Node::shouldSpeculateInt32ForArithmetic(node->child1().node(), node->child2().node())
     268                    && nodeCanSpeculateInt32(node->arithNodeFlags()))
    269269                    changed |= mergePrediction(SpecInt32);
    270270                else
    … …  
    279279           
    280280            if (left && right) {
    281                 if (m_graph.mulShouldSpeculateInteger(node))
     281                if (m_graph.mulShouldSpeculateInt32(node))
    282282                    changed |= mergePrediction(SpecInt32);
    283283                else
    … …  
    292292           
    293293            if (left && right) {
    294                 if (Node::shouldSpeculateIntegerForArithmetic(node->child1().node(), node->child2().node())
    295                     && nodeCanSpeculateInteger(node->arithNodeFlags()))
     294                if (Node::shouldSpeculateInt32ForArithmetic(node->child1().node(), node->child2().node())
     295                    && nodeCanSpeculateInt32(node->arithNodeFlags()))
    296296                    changed |= mergePrediction(SpecInt32);
    297297                else
    … …  
    306306           
    307307            if (left && right) {
    308                 if (Node::shouldSpeculateIntegerForArithmetic(node->child1().node(), node->child2().node())
    309                     && nodeCanSpeculateInteger(node->arithNodeFlags()))
     308                if (Node::shouldSpeculateInt32ForArithmetic(node->child1().node(), node->child2().node())
     309                    && nodeCanSpeculateInt32(node->arithNodeFlags()))
    310310                    changed |= mergePrediction(SpecInt32);
    311311                else
    … …  
    323323            SpeculatedType child = node->child1()->prediction();
    324324            if (isInt32SpeculationForArithmetic(child)
    325                 && nodeCanSpeculateInteger(node->arithNodeFlags()))
     325                && nodeCanSpeculateInt32(node->arithNodeFlags()))
    326326                changed |= mergePrediction(SpecInt32);
    327327            else
    … …  
    629629               
    630630            if (isNumberSpeculationExpectingDefined(left) && isNumberSpeculationExpectingDefined(right)
    631                 && !m_graph.addShouldSpeculateInteger(node))
     631                && !m_graph.addShouldSpeculateInt32(node))
    632632                ballot = VoteDouble;
    633633            else
    … …  
    646646               
    647647            if (isNumberSpeculation(left) && isNumberSpeculation(right)
    648                 && !m_graph.mulShouldSpeculateInteger(node))
     648                && !m_graph.mulShouldSpeculateInt32(node))
    649649                ballot = VoteDouble;
    650650            else
    … …  
    666666               
    667667            if (isNumberSpeculation(left) && isNumberSpeculation(right)
    668                 && !(Node::shouldSpeculateIntegerForArithmetic(node->child1().node(), node->child2().node()) && node->canSpeculateInteger()))
     668                && !(Node::shouldSpeculateInt32ForArithmetic(node->child1().node(), node->child2().node()) && node->canSpeculateInt32()))
    669669                ballot = VoteDouble;
    670670            else
    … …  
    678678        case ArithAbs:
    679679            DoubleBallot ballot;
    680             if (!(node->child1()->shouldSpeculateIntegerForArithmetic() && node->canSpeculateInteger()))
     680            if (!(node->child1()->shouldSpeculateInt32ForArithmetic() && node->canSpeculateInt32()))
    681681                ballot = VoteDouble;
    682682            else
Note: See TracChangeset for help on using the changeset viewer.