Change remaining_bytes API to take const void* instead of address_t

Add a const void* overload of remaining_bytes alongside the existing
address_t version. The testlib API uses const void* so callers don't
need address_cast or ds_aal headers. Remove address_t from testlib.h.

This makes memory.cc a testlib-only test (no ds_aal.h needed),
adding it to the compile-once list.

Author: mjp41

CMakeLists.txt

@@ -483,7 +483,7 @@ if(NOT SNMALLOC_HEADER_ONLY_LIBRARY)
   # These are mitigation-independent and can be compiled once, then linked
   # against both fast and check testlib variants.
   set(TESTLIB_ONLY_TESTS
-    bits first_operation memory_usage multi_atexit multi_threadatexit
+    bits first_operation memory memory_usage multi_atexit multi_threadatexit
     redblack statistics teardown
     contention external_pointer large_alloc lotsofthreads post_teardown
     singlethread startup

src/snmalloc/global/globalalloc.h

@@ -151,6 +151,12 @@ namespace snmalloc
     return snmalloc::remaining_bytes(sizeclass, p);
   }
 
+  template<SNMALLOC_CONCEPT(IsConfig) Config_ = Config>
+  size_t SNMALLOC_FAST_PATH_INLINE remaining_bytes(const void* p)
+  {
+    return remaining_bytes<Config_>(address_cast(p));
+  }
+
   /**
    * Returns the byte offset into an object.
    *

src/test/func/memory/memory.cc

@@ -1,7 +1,6 @@
 #include <array>
 #include <chrono>
 #include <iostream>
-#include <snmalloc/ds_aal/ds_aal.h>
 #include <test/opt.h>
 #include <test/setup.h>
 #include <test/snmalloc_testlib.h>
@@ -494,8 +493,7 @@ void test_remaining_bytes()
     char* p = (char*)snmalloc::alloc(size);
     for (size_t offset = 0; offset < size; offset++)
     {
-      auto rem = snmalloc::remaining_bytes(
-        address_cast(pointer_offset(p, offset)));
+      auto rem = snmalloc::remaining_bytes(pointer_offset(p, offset));
       if (rem != (size - offset))
       {
         if (rem < (size - offset) || snmalloc::is_owned(p))

src/test/snmalloc_testlib.cc

@@ -58,7 +58,7 @@ namespace snmalloc
     return alloc_size<Config>(p);
   }
 
-  size_t remaining_bytes(address_t p)
+  size_t remaining_bytes(const void* p)
   {
     return remaining_bytes<Config>(p);
   }

src/test/snmalloc_testlib.h

@@ -22,10 +22,6 @@
 
 namespace snmalloc
 {
-  // Forward declarations sufficient for the API surface.
-  // address_t: uintptr_t on all non-CHERI platforms.
-  using address_t = uintptr_t;
-
   template<ZeroMem>
   class DefaultConts;
   // Uninit / Zero are usable as template arguments even without the full
@@ -55,7 +51,7 @@ namespace snmalloc
 
   // -- Non-template wrappers for Config-templated functions ----------------
   size_t alloc_size(const void* p);
-  size_t remaining_bytes(address_t p);
+  size_t remaining_bytes(const void* p);
   bool is_owned(void* p);
   void debug_check_empty(bool* result = nullptr);
   void debug_in_use(size_t count);