Rename IntegerOperand to Int32Operand and fillInteger() to fillInt32().
authorfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 12 Sep 2013 05:52:06 +0000 (05:52 +0000)
committerfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 12 Sep 2013 05:52:06 +0000 (05:52 +0000)
Rubber stamped by Mark Hahnenberg.

* dfg/DFGGenerationInfo.h:
(JSC::DFG::GenerationInfo::fillInt32):
* dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::GPRTemporary::GPRTemporary):
(JSC::DFG::SpeculativeJIT::compileUInt32ToNumber):
* dfg/DFGSpeculativeJIT.h:
(JSC::DFG::Int32Operand::Int32Operand):
(JSC::DFG::Int32Operand::~Int32Operand):
(JSC::DFG::Int32Operand::gpr):
* dfg/DFGSpeculativeJIT32_64.cpp:
(JSC::DFG::SpeculativeJIT::fillInt32):
(JSC::DFG::SpeculativeJIT::nonSpeculativeUInt32ToNumber):
(JSC::DFG::SpeculativeJIT::fillSpecualteInt32Internal):
(JSC::DFG::SpeculativeJIT::fillSpeculateDouble):
* dfg/DFGSpeculativeJIT64.cpp:
(JSC::DFG::SpeculativeJIT::fillInt32):
(JSC::DFG::SpeculativeJIT::nonSpeculativeUInt32ToNumber):
(JSC::DFG::SpeculativeJIT::fillSpecualteInt32Internal):
(JSC::DFG::SpeculativeJIT::fillSpeculateDouble):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@155594 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/dfg/DFGGenerationInfo.h
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.h
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp
Source/JavaScriptCore/dfg/DFGSpeculativeJIT64.cpp

index 3e2fd3c..fb53472 100644 (file)
@@ -1,5 +1,31 @@
 2013-09-11  Filip Pizlo  <fpizlo@apple.com>
 
+        Rename IntegerOperand to Int32Operand and fillInteger() to fillInt32().
+
+        Rubber stamped by Mark Hahnenberg.
+
+        * dfg/DFGGenerationInfo.h:
+        (JSC::DFG::GenerationInfo::fillInt32):
+        * dfg/DFGSpeculativeJIT.cpp:
+        (JSC::DFG::GPRTemporary::GPRTemporary):
+        (JSC::DFG::SpeculativeJIT::compileUInt32ToNumber):
+        * dfg/DFGSpeculativeJIT.h:
+        (JSC::DFG::Int32Operand::Int32Operand):
+        (JSC::DFG::Int32Operand::~Int32Operand):
+        (JSC::DFG::Int32Operand::gpr):
+        * dfg/DFGSpeculativeJIT32_64.cpp:
+        (JSC::DFG::SpeculativeJIT::fillInt32):
+        (JSC::DFG::SpeculativeJIT::nonSpeculativeUInt32ToNumber):
+        (JSC::DFG::SpeculativeJIT::fillSpecualteInt32Internal):
+        (JSC::DFG::SpeculativeJIT::fillSpeculateDouble):
+        * dfg/DFGSpeculativeJIT64.cpp:
+        (JSC::DFG::SpeculativeJIT::fillInt32):
+        (JSC::DFG::SpeculativeJIT::nonSpeculativeUInt32ToNumber):
+        (JSC::DFG::SpeculativeJIT::fillSpecualteInt32Internal):
+        (JSC::DFG::SpeculativeJIT::fillSpeculateDouble):
+
+2013-09-11  Filip Pizlo  <fpizlo@apple.com>
+
         FixupPhase should always call fixEdge() exactly once for every edge
         https://bugs.webkit.org/show_bug.cgi?id=121211
 
index 96941cc..d86f11a 100644 (file)
@@ -337,7 +337,7 @@ public:
             appendFill(Fill, stream);
     }
 #endif
-    void fillInteger(VariableEventStream& stream, GPRReg gpr)
+    void fillInt32(VariableEventStream& stream, GPRReg gpr)
     {
         m_registerFormat = DataFormatInt32;
         u.gpr = gpr;
index f7e6af7..f75693a 100644 (file)
@@ -1216,7 +1216,7 @@ GPRTemporary::GPRTemporary(SpeculativeJIT* jit, SpeculateStrictInt32Operand& op1
         m_gpr = m_jit->allocate();
 }
 
-GPRTemporary::GPRTemporary(SpeculativeJIT* jit, IntegerOperand& op1)
+GPRTemporary::GPRTemporary(SpeculativeJIT* jit, Int32Operand& op1)
     : m_jit(jit)
     , m_gpr(InvalidGPRReg)
 {
@@ -1226,7 +1226,7 @@ GPRTemporary::GPRTemporary(SpeculativeJIT* jit, IntegerOperand& op1)
         m_gpr = m_jit->allocate();
 }
 
-GPRTemporary::GPRTemporary(SpeculativeJIT* jit, IntegerOperand& op1, IntegerOperand& op2)
+GPRTemporary::GPRTemporary(SpeculativeJIT* jit, Int32Operand& op1, Int32Operand& op2)
     : m_jit(jit)
     , m_gpr(InvalidGPRReg)
 {
@@ -2408,7 +2408,7 @@ void SpeculativeJIT::compileUInt32ToNumber(Node* node)
         // We know that this sometimes produces doubles. So produce a double every
         // time. This at least allows subsequent code to not have weird conditionals.
             
-        IntegerOperand op1(this, node->child1());
+        Int32Operand op1(this, node->child1());
         FPRTemporary result(this);
             
         GPRReg inputGPR = op1.gpr();
@@ -2424,7 +2424,7 @@ void SpeculativeJIT::compileUInt32ToNumber(Node* node)
         return;
     }
 
-    IntegerOperand op1(this, node->child1());
+    Int32Operand op1(this, node->child1());
     GPRTemporary result(this); // For the benefit of OSR exit, force these to be in different registers. In reality the OSR exit compiler could find cases where you have uint32(%r1) followed by int32(%r1) and then use different registers, but that seems like too much effort.
 
     m_jit.move(op1.gpr(), result.gpr());
index 7d9a911..e150999 100644 (file)
@@ -130,7 +130,7 @@ public:
         }
     }
     
-    GPRReg fillInteger(Edge, DataFormat& returnFormat);
+    GPRReg fillInt32(Edge, DataFormat& returnFormat);
 #if USE(JSVALUE64)
     GPRReg fillJSValue(Edge);
 #elif USE(JSVALUE32_64)
@@ -2247,7 +2247,7 @@ public:
 
 // === Operand types ===
 //
-// IntegerOperand and JSValueOperand.
+// Int32Operand and JSValueOperand.
 //
 // These classes are used to lock the operands to a node into machine
 // registers. These classes implement of pattern of locking a value
@@ -2256,9 +2256,9 @@ public:
 // used. We do so in order to attempt to avoid spilling one operand
 // in order to make space available for another.
 
-class IntegerOperand {
+class Int32Operand {
 public:
-    explicit IntegerOperand(SpeculativeJIT* jit, Edge edge, OperandSpeculationMode mode = AutomaticOperandSpeculation)
+    explicit Int32Operand(SpeculativeJIT* jit, Edge edge, OperandSpeculationMode mode = AutomaticOperandSpeculation)
         : m_jit(jit)
         , m_edge(edge)
         , m_gprOrInvalid(InvalidGPRReg)
@@ -2272,7 +2272,7 @@ public:
             gpr();
     }
 
-    ~IntegerOperand()
+    ~Int32Operand()
     {
         ASSERT(m_gprOrInvalid != InvalidGPRReg);
         m_jit->unlock(m_gprOrInvalid);
@@ -2298,7 +2298,7 @@ public:
     GPRReg gpr()
     {
         if (m_gprOrInvalid == InvalidGPRReg)
-            m_gprOrInvalid = m_jit->fillInteger(m_edge, m_format);
+            m_gprOrInvalid = m_jit->fillInt32(m_edge, m_format);
         return m_gprOrInvalid;
     }
     
@@ -2497,8 +2497,8 @@ public:
     GPRTemporary(SpeculativeJIT*, SpeculateInt32Operand&);
     GPRTemporary(SpeculativeJIT*, SpeculateInt32Operand&, SpeculateInt32Operand&);
     GPRTemporary(SpeculativeJIT*, SpeculateStrictInt32Operand&);
-    GPRTemporary(SpeculativeJIT*, IntegerOperand&);
-    GPRTemporary(SpeculativeJIT*, IntegerOperand&, IntegerOperand&);
+    GPRTemporary(SpeculativeJIT*, Int32Operand&);
+    GPRTemporary(SpeculativeJIT*, Int32Operand&, Int32Operand&);
     GPRTemporary(SpeculativeJIT*, SpeculateCellOperand&);
     GPRTemporary(SpeculativeJIT*, SpeculateBooleanOperand&);
 #if USE(JSVALUE64)
index e6917d2..ab1f544 100644 (file)
@@ -41,7 +41,7 @@ namespace JSC { namespace DFG {
 
 #if USE(JSVALUE32_64)
 
-GPRReg SpeculativeJIT::fillInteger(Edge edge, DataFormat& returnFormat)
+GPRReg SpeculativeJIT::fillInt32(Edge edge, DataFormat& returnFormat)
 {
     ASSERT(!needsTypeCheck(edge, SpecInt32));
     
@@ -68,7 +68,7 @@ GPRReg SpeculativeJIT::fillInteger(Edge edge, DataFormat& returnFormat)
             m_jit.load32(JITCompiler::payloadFor(virtualRegister), gpr);
         }
 
-        info.fillInteger(*m_stream, gpr);
+        info.fillInt32(*m_stream, gpr);
         returnFormat = DataFormatInt32;
         return gpr;
     }
@@ -97,7 +97,7 @@ GPRReg SpeculativeJIT::fillInteger(Edge edge, DataFormat& returnFormat)
         m_gprs.release(tagGPR);
         m_gprs.release(payloadGPR);
         m_gprs.retain(payloadGPR, virtualRegister, SpillOrderInteger);
-        info.fillInteger(*m_stream, payloadGPR);
+        info.fillInt32(*m_stream, payloadGPR);
         returnFormat = DataFormatInt32;
         return payloadGPR;
     }
@@ -244,7 +244,7 @@ bool SpeculativeJIT::fillJSValue(Edge edge, GPRReg& tagGPR, GPRReg& payloadGPR,
 
 void SpeculativeJIT::nonSpeculativeUInt32ToNumber(Node* node)
 {
-    IntegerOperand op1(this, node->child1());
+    Int32Operand op1(this, node->child1());
     FPRTemporary boxer(this);
     GPRTemporary resultTag(this, op1);
     GPRTemporary resultPayload(this);
@@ -880,7 +880,7 @@ GPRReg SpeculativeJIT::fillSpecualteInt32Internal(Edge edge, DataFormat& returnF
             GPRReg gpr = allocate();
             m_jit.move(MacroAssembler::Imm32(valueOfInt32Constant(edge.node())), gpr);
             m_gprs.retain(gpr, virtualRegister, SpillOrderConstant);
-            info.fillInteger(*m_stream, gpr);
+            info.fillInt32(*m_stream, gpr);
             returnFormat = DataFormatInt32;
             return gpr;
         }
@@ -895,7 +895,7 @@ GPRReg SpeculativeJIT::fillSpecualteInt32Internal(Edge edge, DataFormat& returnF
         GPRReg gpr = allocate();
         m_jit.load32(JITCompiler::payloadFor(virtualRegister), gpr);
         m_gprs.retain(gpr, virtualRegister, SpillOrderSpilled);
-        info.fillInteger(*m_stream, gpr);
+        info.fillInt32(*m_stream, gpr);
         returnFormat = DataFormatInt32;
         return gpr;
     }
@@ -913,7 +913,7 @@ GPRReg SpeculativeJIT::fillSpecualteInt32Internal(Edge edge, DataFormat& returnF
         m_gprs.release(tagGPR);
         m_gprs.release(payloadGPR);
         m_gprs.retain(payloadGPR, virtualRegister, SpillOrderInteger);
-        info.fillInteger(*m_stream, payloadGPR);
+        info.fillInt32(*m_stream, payloadGPR);
         // If !strict we're done, return.
         returnFormat = DataFormatInt32;
         return payloadGPR;
@@ -975,7 +975,7 @@ FPRReg SpeculativeJIT::fillSpeculateDouble(Edge edge)
                 GPRReg gpr = allocate();
                 m_jit.move(MacroAssembler::Imm32(valueOfInt32Constant(edge.node())), gpr);
                 m_gprs.retain(gpr, virtualRegister, SpillOrderConstant);
-                info.fillInteger(*m_stream, gpr);
+                info.fillInt32(*m_stream, gpr);
                 unlock(gpr);
             } else if (isNumberConstant(edge.node())) {
                 FPRReg fpr = fprAllocate();
index 674108c..f5ffc39 100644 (file)
@@ -40,7 +40,7 @@ namespace JSC { namespace DFG {
 
 #if USE(JSVALUE64)
 
-GPRReg SpeculativeJIT::fillInteger(Edge edge, DataFormat& returnFormat)
+GPRReg SpeculativeJIT::fillInt32(Edge edge, DataFormat& returnFormat)
 {
     ASSERT(!needsTypeCheck(edge, SpecInt32));
     
@@ -54,7 +54,7 @@ GPRReg SpeculativeJIT::fillInteger(Edge edge, DataFormat& returnFormat)
             m_gprs.retain(gpr, virtualRegister, SpillOrderConstant);
             if (isInt32Constant(edge.node())) {
                 m_jit.move(MacroAssembler::Imm32(valueOfInt32Constant(edge.node())), gpr);
-                info.fillInteger(*m_stream, gpr);
+                info.fillInt32(*m_stream, gpr);
                 returnFormat = DataFormatInt32;
                 return gpr;
             }
@@ -69,7 +69,7 @@ GPRReg SpeculativeJIT::fillInteger(Edge edge, DataFormat& returnFormat)
         } else if (info.spillFormat() == DataFormatInt32) {
             m_gprs.retain(gpr, virtualRegister, SpillOrderSpilled);
             m_jit.load32(JITCompiler::payloadFor(virtualRegister), gpr);
-            // Tag it, since fillInteger() is used when we want a boxed integer.
+            // Tag it, since fillInt32() is used when we want a boxed integer.
             m_jit.or64(GPRInfo::tagTypeNumberRegister, gpr);
         } else {
             RELEASE_ASSERT(info.spillFormat() == DataFormatJS || info.spillFormat() == DataFormatJSInt32);
@@ -219,7 +219,7 @@ GPRReg SpeculativeJIT::fillJSValue(Edge edge)
 
 void SpeculativeJIT::nonSpeculativeUInt32ToNumber(Node* node)
 {
-    IntegerOperand op1(this, node->child1());
+    Int32Operand op1(this, node->child1());
     FPRTemporary boxer(this);
     GPRTemporary result(this, op1);
     
@@ -841,7 +841,7 @@ GPRReg SpeculativeJIT::fillSpecualteInt32Internal(Edge edge, DataFormat& returnF
             m_gprs.retain(gpr, virtualRegister, SpillOrderConstant);
             ASSERT(isInt32Constant(edge.node()));
             m_jit.move(MacroAssembler::Imm32(valueOfInt32Constant(edge.node())), gpr);
-            info.fillInteger(*m_stream, gpr);
+            info.fillInt32(*m_stream, gpr);
             returnFormat = DataFormatInt32;
             return gpr;
         }
@@ -856,7 +856,7 @@ GPRReg SpeculativeJIT::fillSpecualteInt32Internal(Edge edge, DataFormat& returnF
             // If we know this was spilled as an integer we can fill without checking.
             if (strict) {
                 m_jit.load32(JITCompiler::addressFor(virtualRegister), gpr);
-                info.fillInteger(*m_stream, gpr);
+                info.fillInt32(*m_stream, gpr);
                 returnFormat = DataFormatInt32;
                 return gpr;
             }
@@ -903,7 +903,7 @@ GPRReg SpeculativeJIT::fillSpecualteInt32Internal(Edge edge, DataFormat& returnF
                 result = allocate();
             else {
                 m_gprs.lock(gpr);
-                info.fillInteger(*m_stream, gpr);
+                info.fillInt32(*m_stream, gpr);
                 result = gpr;
             }
             m_jit.zeroExtend32ToPtr(gpr, result);
@@ -1021,7 +1021,7 @@ FPRReg SpeculativeJIT::fillSpeculateDouble(Edge edge)
             
             m_gprs.retain(gpr, virtualRegister, SpillOrderSpilled);
             m_jit.load32(JITCompiler::addressFor(virtualRegister), gpr);
-            info.fillInteger(*m_stream, gpr);
+            info.fillInt32(*m_stream, gpr);
             unlock(gpr);
             break;
         }