Title: [176425] trunk/Source/_javascript_Core
Revision
176425
Author
[email protected]
Date
2014-11-20 15:39:46 -0800 (Thu, 20 Nov 2014)

Log Message

Add some comments to describe the DFG UseKind representations.
<https://webkit.org/b/138934>

Reviewed by Filip Pizlo.

* dfg/DFGUseKind.h:
- Also regrouped the UseKind enums by representation to be more readable.

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (176424 => 176425)


--- trunk/Source/_javascript_Core/ChangeLog	2014-11-20 23:28:41 UTC (rev 176424)
+++ trunk/Source/_javascript_Core/ChangeLog	2014-11-20 23:39:46 UTC (rev 176425)
@@ -1,5 +1,15 @@
 2014-11-20  Mark Lam  <[email protected]>
 
+        Add some comments to describe the DFG UseKind representations.
+        <https://webkit.org/b/138934>
+
+        Reviewed by Filip Pizlo.
+
+        * dfg/DFGUseKind.h:
+        - Also regrouped the UseKind enums by representation to be more readable.
+
+2014-11-20  Mark Lam  <[email protected]>
+
         Add Heap verification infrastructure.
         <https://webkit.org/b/138851>
 

Modified: trunk/Source/_javascript_Core/dfg/DFGUseKind.h (176424 => 176425)


--- trunk/Source/_javascript_Core/dfg/DFGUseKind.h	2014-11-20 23:28:41 UTC (rev 176424)
+++ trunk/Source/_javascript_Core/dfg/DFGUseKind.h	2014-11-20 23:39:46 UTC (rev 176425)
@@ -35,15 +35,17 @@
 namespace JSC { namespace DFG {
 
 enum UseKind {
-    UntypedUse,
+    // The DFG has 3 representations of values used:
+
+    // 1. The JSValue representation for a JSValue that must be stored in a GP
+    //    register (or a GP register pair), and follows rules for boxing and unboxing
+    //    that allow the JSValue to be stored as either fully boxed JSValues, or
+    //    unboxed Int32, Booleans, Cells, etc. in 32-bit as appropriate.
+    UntypedUse, // UntypedUse must come first (value 0).
     Int32Use,
     KnownInt32Use,
-    Int52RepUse,
     MachineIntUse,
     NumberUse,
-    DoubleRepUse,
-    DoubleRepRealUse,
-    DoubleRepMachineIntUse,
     BooleanUse,
     CellUse,
     KnownCellUse,
@@ -60,6 +62,17 @@
     NotCellUse,
     OtherUse,
     MiscUse,
+
+    // 2. The Double representation for an unboxed double value that must be stored
+    //    in an FP register.
+    DoubleRepUse,
+    DoubleRepRealUse,
+    DoubleRepMachineIntUse,
+
+    // 3. The Int52 representation for an unboxed integer value that must be stored
+    //    in a GP register.
+    Int52RepUse,
+
     LastUseKind // Must always be the last entry in the enum, as it is used to denote the number of enum elements.
 };
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to