Rename DFG's compileAdd to compileArithAdd.
authormark.lam@apple.com <mark.lam@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 4 Nov 2015 01:00:39 +0000 (01:00 +0000)
committermark.lam@apple.com <mark.lam@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 4 Nov 2015 01:00:39 +0000 (01:00 +0000)
https://bugs.webkit.org/show_bug.cgi?id=150866

Reviewed by Benjamin Poulain.

The function is only supposed to generate code to do arithmetic addition on
numeric types.  Naming it compileArithAdd() is more accurate, and is consistent
with the name of the node it emits code for (i.e. ArithAdd) as well as other
compiler functions for analogous operations e.g. compileArithSub.

* dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculativeJIT::compileInstanceOf):
(JSC::DFG::SpeculativeJIT::compileArithAdd):
(JSC::DFG::SpeculativeJIT::compileAdd): Deleted.
* dfg/DFGSpeculativeJIT.h:
* dfg/DFGSpeculativeJIT32_64.cpp:
(JSC::DFG::SpeculativeJIT::compile):
* dfg/DFGSpeculativeJIT64.cpp:
(JSC::DFG::SpeculativeJIT::compile):

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

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

index e172cd41ffd8b78ba1077af19aa1571736d083fb..8023dd3a01661c70d450e5709c4df3c1d4bf37a1 100644 (file)
@@ -1,3 +1,25 @@
+2015-11-03  Mark Lam  <mark.lam@apple.com>
+
+        Rename DFG's compileAdd to compileArithAdd.
+        https://bugs.webkit.org/show_bug.cgi?id=150866
+
+        Reviewed by Benjamin Poulain.
+
+        The function is only supposed to generate code to do arithmetic addition on
+        numeric types.  Naming it compileArithAdd() is more accurate, and is consistent
+        with the name of the node it emits code for (i.e. ArithAdd) as well as other
+        compiler functions for analogous operations e.g. compileArithSub.
+
+        * dfg/DFGSpeculativeJIT.cpp:
+        (JSC::DFG::SpeculativeJIT::compileInstanceOf):
+        (JSC::DFG::SpeculativeJIT::compileArithAdd):
+        (JSC::DFG::SpeculativeJIT::compileAdd): Deleted.
+        * dfg/DFGSpeculativeJIT.h:
+        * dfg/DFGSpeculativeJIT32_64.cpp:
+        (JSC::DFG::SpeculativeJIT::compile):
+        * dfg/DFGSpeculativeJIT64.cpp:
+        (JSC::DFG::SpeculativeJIT::compile):
+
 2015-11-03  Joseph Pecoraro  <pecoraro@apple.com>
 
         Web Inspector: Remove duplication among ScriptDebugServer subclasses
 2015-11-03  Joseph Pecoraro  <pecoraro@apple.com>
 
         Web Inspector: Remove duplication among ScriptDebugServer subclasses
index 06e5484fc9b74d617ef26c164ef0db20b8263680..1d9cd448f62aaeaa90e8fb207efb6fd64818d808 100755 (executable)
@@ -2782,7 +2782,7 @@ void SpeculativeJIT::compileInstanceOf(Node* node)
     blessedBooleanResult(scratchReg, node);
 }
 
     blessedBooleanResult(scratchReg, node);
 }
 
-void SpeculativeJIT::compileAdd(Node* node)
+void SpeculativeJIT::compileArithAdd(Node* node)
 {
     switch (node->binaryUseKind()) {
     case Int32Use: {
 {
     switch (node->binaryUseKind()) {
     case Int32Use: {
index 47250b36eec66e6d23672641ca44fceb24b9826c..4b835ef8ccb4cb14a6fc1505eee54dcf3050a525 100755 (executable)
@@ -2207,7 +2207,7 @@ public:
     void compileValueToInt32(Node*);
     void compileUInt32ToNumber(Node*);
     void compileDoubleAsInt32(Node*);
     void compileValueToInt32(Node*);
     void compileUInt32ToNumber(Node*);
     void compileDoubleAsInt32(Node*);
-    void compileAdd(Node*);
+    void compileArithAdd(Node*);
     void compileMakeRope(Node*);
     void compileArithClz32(Node*);
     void compileArithSub(Node*);
     void compileMakeRope(Node*);
     void compileArithClz32(Node*);
     void compileArithSub(Node*);
index 4d6908fc9a601a668de2816039e3604b95f06bd6..c76b8b50d68323015e1a76c01ec3d877fcb6a838 100644 (file)
@@ -2191,7 +2191,7 @@ void SpeculativeJIT::compile(Node* node)
     }
 
     case ArithAdd:
     }
 
     case ArithAdd:
-        compileAdd(node);
+        compileArithAdd(node);
         break;
 
     case ArithClz32:
         break;
 
     case ArithClz32:
index d0b750901e57e3e986a19b56b2976e51ca2b6fde..5e13b5b1e6e1685bb16e85bc697887bab0d2c003 100644 (file)
@@ -2318,7 +2318,7 @@ void SpeculativeJIT::compile(Node* node)
     }
 
     case ArithAdd:
     }
 
     case ArithAdd:
-        compileAdd(node);
+        compileArithAdd(node);
         break;
 
     case ArithClz32:
         break;
 
     case ArithClz32: