fix windows build errors.

Author: mjp41

src/snmalloc/mem/largecache.h

@@ -160,14 +160,14 @@ namespace snmalloc
      */
     void ensure_registered()
     {
-      if constexpr (!pal_supports<Time, PAL>)
-        return;
-
-      if (
-        !registered_timer.load(stl::memory_order_relaxed) &&
-        !registered_timer.exchange(true, stl::memory_order_acq_rel))
+      if constexpr (pal_supports<Time, PAL>)
       {
-        PAL::register_timer(&timer_object);
+        if (
+          !registered_timer.load(stl::memory_order_relaxed) &&
+          !registered_timer.exchange(true, stl::memory_order_acq_rel))
+        {
+          PAL::register_timer(&timer_object);
+        }
       }
     }
 
@@ -178,54 +178,54 @@ namespace snmalloc
     template<typename FlushFn>
     void sync_epoch(FlushFn&& flush_fn)
     {
-      if constexpr (!pal_supports<Time, PAL>)
-        return;
-
-      auto current = global_epoch.load(stl::memory_order_acquire);
-
-      auto behind = current - local_epoch;
-      if (behind == 0)
-        return;
+      if constexpr (pal_supports<Time, PAL>)
+      {
+        auto current = global_epoch.load(stl::memory_order_acquire);
 
-      if (behind > NUM_EPOCHS)
-        behind = NUM_EPOCHS;
+        auto behind = current - local_epoch;
+        if (behind == 0)
+          return;
 
-      // Snapshot counts before flushing.
-      size_t before_count[NUM_SIZECLASSES];
-      for (size_t sc = 0; sc < NUM_SIZECLASSES; sc++)
-        before_count[sc] = sc_state[sc].count;
+        if (behind > NUM_EPOCHS)
+          behind = NUM_EPOCHS;
 
-      // Flush stale epoch slots.
-      for (size_t i = 0; i < behind; i++)
-      {
-        auto epoch_to_flush = (local_epoch + 1 + i) % NUM_EPOCHS;
-        flush_epoch_slot(epoch_to_flush, flush_fn);
-      }
+        // Snapshot counts before flushing.
+        size_t before_count[NUM_SIZECLASSES];
+        for (size_t sc = 0; sc < NUM_SIZECLASSES; sc++)
+          before_count[sc] = sc_state[sc].count;
 
-      // Adjust budgets based on what happened.
-      // Net out misses against flushed items to determine direction.
-      for (size_t sc = 0; sc < NUM_SIZECLASSES; sc++)
-      {
-        auto& state = sc_state[sc];
-        size_t flushed = before_count[sc] - state.count;
-
-        if (state.peak_misses > flushed)
+        // Flush stale epoch slots.
+        for (size_t i = 0; i < behind; i++)
         {
-          // More misses than surplus: grow budget by the difference.
-          state.budget += state.peak_misses - flushed;
+          auto epoch_to_flush = (local_epoch + 1 + i) % NUM_EPOCHS;
+          flush_epoch_slot(epoch_to_flush, flush_fn);
         }
-        else if (flushed > state.peak_misses)
+
+        // Adjust budgets based on what happened.
+        // Net out misses against flushed items to determine direction.
+        for (size_t sc = 0; sc < NUM_SIZECLASSES; sc++)
         {
-          // More surplus than misses: shrink budget smoothly.
-          state.budget -= (flushed - state.peak_misses) / 2;
+          auto& state = sc_state[sc];
+          size_t flushed = before_count[sc] - state.count;
+
+          if (state.peak_misses > flushed)
+          {
+            // More misses than surplus: grow budget by the difference.
+            state.budget += state.peak_misses - flushed;
+          }
+          else if (flushed > state.peak_misses)
+          {
+            // More surplus than misses: shrink budget smoothly.
+            state.budget -= (flushed - state.peak_misses) / 2;
+          }
+          // If equal, budget stays the same.
+
+          state.misses = 0;
+          state.peak_misses = 0;
         }
-        // If equal, budget stays the same.
 
-        state.misses = 0;
-        state.peak_misses = 0;
+        local_epoch = current;
       }
-
-      local_epoch = current;
     }
 
     /**

src/test/perf/large_alloc/large_alloc.cc

@@ -32,7 +32,7 @@ void test_batch_alloc_then_dealloc()
 
   MeasureTime m;
   m << "Batch alloc then dealloc 800KB x " << BATCH;
-  for (size_t i = 0; i < ITERATIONS / BATCH; i++)
+  for (size_t j = 0; j < ITERATIONS / BATCH; j++)
   {
     for (size_t i = 0; i < BATCH; i++)
     {