Bug 983959 - Part 1: Fix warnings about unused member variables in js/. r=luke

js/src/assembler/jit/ExecutableAllocator.h

@@ -115,13 +115,6 @@ private:
     size_t m_otherCodeBytes;
 
 public:
-    // Flag for downstream use, whether to try to release references to this pool.
-    bool m_destroy;
-
-    // GC number in which the m_destroy flag was most recently set. Used downstream to
-    // remember whether m_destroy was computed for the currently active GC.
-    size_t m_gcNumber;
-
     void release(bool willDestroy = false)
     {
         JS_ASSERT(m_refCount != 0);
@@ -159,7 +152,7 @@ public:
     ExecutablePool(ExecutableAllocator* allocator, Allocation a)
       : m_allocator(allocator), m_freePtr(a.pages), m_end(m_freePtr + a.size), m_allocation(a),
         m_refCount(1), m_ionCodeBytes(0), m_baselineCodeBytes(0), m_regexpCodeBytes(0),
-        m_otherCodeBytes(0), m_destroy(false), m_gcNumber(0)
+        m_otherCodeBytes(0)
     { }
 
     ~ExecutablePool();

js/src/vm/ThreadPool.cpp

@@ -238,11 +238,11 @@ ThreadPoolWorker::terminate(AutoLockMonitor &lock)
 // them down when requested.
 
 ThreadPool::ThreadPool(JSRuntime *rt)
-  : runtime_(rt),
-    activeWorkers_(0),
+  : activeWorkers_(0),
     joinBarrier_(nullptr),
     job_(nullptr),
 #ifdef DEBUG
+    runtime_(rt),
     stolenSlices_(0),
 #endif
     pendingSlices_(0),

js/src/vm/ThreadPool.h

@@ -164,9 +164,6 @@ class ThreadPool : public Monitor
   private:
     friend class ThreadPoolWorker;
 
-    // Initialized at startup only.
-    JSRuntime *const runtime_;
-
     // Initialized lazily.
     js::Vector<ThreadPoolWorker *, 8, SystemAllocPolicy> workers_;
 
@@ -178,6 +175,9 @@ class ThreadPool : public Monitor
     ParallelJob *job_;
 
 #ifdef DEBUG
+    // Initialized at startup only.
+    JSRuntime *const runtime_;
+
     // Number of stolen slices in the last parallel job.
     mozilla::Atomic<uint32_t, mozilla::ReleaseAcquire> stolenSlices_;
 #endif