Title: [242944] trunk/Source/_javascript_Core
Revision
242944
Author
[email protected]
Date
2019-03-14 09:41:50 -0700 (Thu, 14 Mar 2019)

Log Message

DFG::Worklist can be shrunk by 16 bytes
https://bugs.webkit.org/show_bug.cgi?id=195490

Reviewed by Darin Adler.

* dfg/DFGWorklist.cpp:
(JSC::DFG::Worklist::Worklist):
* dfg/DFGWorklist.h:

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (242943 => 242944)


--- trunk/Source/_javascript_Core/ChangeLog	2019-03-14 16:21:34 UTC (rev 242943)
+++ trunk/Source/_javascript_Core/ChangeLog	2019-03-14 16:41:50 UTC (rev 242944)
@@ -1,3 +1,14 @@
+2019-03-14  Robin Morisset  <[email protected]>
+
+        DFG::Worklist can be shrunk by 16 bytes
+        https://bugs.webkit.org/show_bug.cgi?id=195490
+
+        Reviewed by Darin Adler.
+
+        * dfg/DFGWorklist.cpp:
+        (JSC::DFG::Worklist::Worklist):
+        * dfg/DFGWorklist.h:
+
 2019-03-14  Devin Rousso  <[email protected]>
 
         Web Inspector: Audit: provide a way to get the contents of resources

Modified: trunk/Source/_javascript_Core/dfg/DFGWorklist.cpp (242943 => 242944)


--- trunk/Source/_javascript_Core/dfg/DFGWorklist.cpp	2019-03-14 16:21:34 UTC (rev 242943)
+++ trunk/Source/_javascript_Core/dfg/DFGWorklist.cpp	2019-03-14 16:41:50 UTC (rev 242944)
@@ -185,9 +185,8 @@
 
 Worklist::Worklist(CString&& tierName)
     : m_threadName(createWorklistName(WTFMove(tierName)))
+    , m_planEnqueued(AutomaticThreadCondition::create())
     , m_lock(Box<Lock>::create())
-    , m_planEnqueued(AutomaticThreadCondition::create())
-    , m_numberOfActiveThreads(0)
 {
 }
 

Modified: trunk/Source/_javascript_Core/dfg/DFGWorklist.h (242943 => 242944)


--- trunk/Source/_javascript_Core/dfg/DFGWorklist.h	2019-03-14 16:21:34 UTC (rev 242943)
+++ trunk/Source/_javascript_Core/dfg/DFGWorklist.h	2019-03-14 16:41:50 UTC (rev 242944)
@@ -96,7 +96,9 @@
 
     void dump(const AbstractLocker&, PrintStream&) const;
     
+    unsigned m_numberOfActiveThreads { 0 };
     CString m_threadName;
+    Vector<std::unique_ptr<ThreadData>> m_threads;
     
     // Used to inform the thread about what work there is left to do.
     Deque<RefPtr<Plan>> m_queue;
@@ -111,15 +113,11 @@
     // Used to quickly find which plans have been compiled and are ready to
     // be completed.
     Vector<RefPtr<Plan>, 16> m_readyPlans;
+    Ref<AutomaticThreadCondition> m_planEnqueued;
+    Condition m_planCompiled;
 
     Lock m_suspensionLock;
-    
     Box<Lock> m_lock;
-    Ref<AutomaticThreadCondition> m_planEnqueued;
-    Condition m_planCompiled;
-    
-    Vector<std::unique_ptr<ThreadData>> m_threads;
-    unsigned m_numberOfActiveThreads;
 };
 
 JS_EXPORT_PRIVATE unsigned setNumberOfDFGCompilerThreads(unsigned);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to