chore: Remove extraneous comments and script output prefixes from scripts and benchmark code.

Author: oleglazari

run_tests.sh

@@ -7,7 +7,6 @@ set -e
 
 BASE_DIR=$(pwd)
 ALLOC_DIR="$BASE_DIR/allocators"
-# Argument parsing
 FILTER_ALLOC=""
 TEST_ARGS=""
 
@@ -42,7 +41,6 @@ run_test() {
     local extra_cflags=$3
     local extra_ldflags=$4
 
-    # Skip if we are filtering for a different allocator
     if [ ! -z "$FILTER_ALLOC" ] && [ "$FILTER_ALLOC" != "$name" ]; then
         return 0
     fi
@@ -97,9 +95,7 @@ if [ -z "$CURRENT_CUSTOM_ALLOC" ]; then
 fi
 
 if [ -f "$CURRENT_CUSTOM_ALLOC" ]; then
-    # We allow "custom" or "skeleton" as filters for this slot
     if [ ! -z "$FILTER_ALLOC" ] && [ "$FILTER_ALLOC" != "custom" ] && [ "$FILTER_ALLOC" != "skeleton" ] && [ "$FILTER_ALLOC" != "$CURRENT_CUSTOM_NAME" ]; then
-        # Skip
         :
     else
         run_test "$CURRENT_CUSTOM_NAME" "$CURRENT_CUSTOM_ALLOC" "$CUSTOM_CFLAGS" "$CUSTOM_LDFLAGS"

setup_allocators.sh

@@ -7,28 +7,28 @@ set -e
 BASE_DIR=$(pwd)
 ALLOC_DIR="$BASE_DIR/allocators"
 
-echo "[*] Checking for build dependencies..."
+echo "Checking for build dependencies..."
 for cmd in git cmake autoconf automake make cc; do
     if ! command -v $cmd &> /dev/null; then
-        echo "[!] Error: $cmd is not installed."
+        echo "Error: $cmd is not installed."
         exit 1
     fi
 done
 
-echo "[*] Setting up dependencies in $ALLOC_DIR..."
+echo "Setting up dependencies in $ALLOC_DIR..."
 mkdir -p "$ALLOC_DIR"
 
 
 MIMALLOC_DIR="$ALLOC_DIR/mimalloc/mimalloc_src"
 if [ ! -d "$MIMALLOC_DIR" ]; then
-    echo "[*] Cloning mimalloc..."
+    echo "Cloning mimalloc..."
     git clone https://github.com/microsoft/mimalloc "$MIMALLOC_DIR"
 else
-    echo "[*] mimalloc already cloned."
+    echo "mimalloc already cloned."
 fi
 
 # Build Mimalloc (Secure/Debug)
-echo "[*] Building mimalloc (secure)..."
+echo "Building mimalloc (secure)..."
 mkdir -p "$BASE_DIR/build_secure"
 cd "$BASE_DIR/build_secure"
 cmake "$MIMALLOC_DIR" -DMI_SECURE=ON -DMI_BUILD_SHARED=OFF -DMI_BUILD_TESTS=OFF
@@ -39,14 +39,14 @@ JEMALLOC_DIR="$ALLOC_DIR/jemalloc/jemalloc_src"
 mkdir -p "$ALLOC_DIR/jemalloc"
 
 if [ ! -d "$JEMALLOC_DIR" ]; then
-    echo "[*] Cloning jemalloc..."
+    echo "Cloning jemalloc..."
     git clone https://github.com/jemalloc/jemalloc "$JEMALLOC_DIR"
 else
-    echo "[*] jemalloc already cloned."
+    echo "jemalloc already cloned."
 fi
 
 # Build Jemalloc (Static, Prefixed)
-echo "[*] Building jemalloc..."
+echo "Building jemalloc..."
 cd "$JEMALLOC_DIR"
 if [ ! -f "configure" ]; then
     ./autogen.sh
@@ -60,6 +60,5 @@ fi
 make -j$(nproc)
 cd "$BASE_DIR"
 
-echo "[+] Setup complete! Libraries built."
 echo "    - Mimalloc: build_secure/libmimalloc-secure.a"
 echo "    - Jemalloc: allocators/jemalloc/jemalloc_src/lib/libjemalloc.a"

src/benchmarks/bench_synthetic.c

@@ -12,7 +12,6 @@ static void wl_syn_001_run(allocator_t *alloc, bench_metrics_t *metrics) {
   latency_samples_t lat;
   latency_init(&lat);
 
-  // Warmup
   for (size_t i = 0; i < BENCH_WARMUP_OPS; i++) {
     void *p = alloc->malloc(64);
     alloc->free(p);
@@ -49,14 +48,13 @@ static void wl_syn_002_run(allocator_t *alloc, bench_metrics_t *metrics) {
   bench_rng_t rng;
   bench_rng_seed(&rng, 0x12345678);
 
-  // Warmup
   for (size_t i = 0; i < BENCH_WARMUP_OPS; i++) {
     size_t sz = bench_rng_range(&rng, 16, 256);
     void *p = alloc->malloc(sz);
     alloc->free(p);
   }
 
-  bench_rng_seed(&rng, 0x12345678); // Reset for reproducibility
+  bench_rng_seed(&rng, 0x12345678);
   uint64_t start = bench_get_time_ns();
 
   for (size_t i = 0; i < iterations; i++) {
@@ -151,7 +149,7 @@ static void wl_syn_005_run(allocator_t *alloc, bench_metrics_t *metrics) {
   latency_samples_t lat;
   latency_init(&lat);
 
-  for (size_t i = 0; i < 1000; i++) { // Smaller warmup for large allocs
+  for (size_t i = 0; i < 1000; i++) {
     void *p = alloc->malloc(1048576);
     alloc->free(p);
   }
@@ -163,7 +161,7 @@ static void wl_syn_005_run(allocator_t *alloc, bench_metrics_t *metrics) {
     void *p = alloc->malloc(1048576);
     alloc->free(p);
 
-    if (i % 10 == 0) { // Sample more frequently for fewer iterations
+    if (i % 10 == 0) {
       latency_record(&lat, bench_get_time_ns() - op_start);
     }
   }
@@ -227,9 +225,7 @@ static void wl_syn_007_run(allocator_t *alloc, bench_metrics_t *metrics) {
   size_t total_ops = 0;
 
   while (total_ops < iterations) {
-    // Allocate batch
     for (size_t i = 0; i < batch_size && total_ops < iterations; i++) {
-      // Power-law: mostly small, some large
       size_t sz = bench_rng_powerlaw(&rng, 16, 65536, 2.0);
       uint64_t op_start = bench_get_time_ns();
       batch[i] = alloc->malloc(sz);
@@ -240,7 +236,6 @@ static void wl_syn_007_run(allocator_t *alloc, bench_metrics_t *metrics) {
       total_ops++;
     }
 
-    // Free batch
     for (size_t i = 0; i < batch_size; i++) {
       if (batch[i]) {
         alloc->free(batch[i]);
@@ -268,7 +263,6 @@ static void wl_syn_008_run(allocator_t *alloc, bench_metrics_t *metrics) {
   for (size_t i = 0; i < iterations; i++) {
     void *p = alloc->malloc(16);
 
-    // Grow: 16 → 32 → 64 → 128 → 256 → 512 → 1024 → 2048 → 4096
     for (size_t sz = 32; sz <= 4096; sz *= 2) {
       uint64_t op_start = bench_get_time_ns();
       p = alloc->realloc(p, sz);
@@ -282,7 +276,6 @@ static void wl_syn_008_run(allocator_t *alloc, bench_metrics_t *metrics) {
   }
 
   uint64_t elapsed = bench_get_time_ns() - start;
-  // Count each realloc chain as 8 operations
   metrics->throughput_ops_sec =
       (double)(iterations * 8) / ((double)elapsed / 1e9);
   metrics->rss_bytes = bench_get_rss();
@@ -302,7 +295,6 @@ static void wl_syn_009_run(allocator_t *alloc, bench_metrics_t *metrics) {
   for (size_t i = 0; i < iterations; i++) {
     void *p = alloc->malloc(4096);
 
-    // Shrink: 4096 → 2048 → 1024 → 512 → 256 → 128 → 64 → 32 → 16
     for (size_t sz = 2048; sz >= 16; sz /= 2) {
       uint64_t op_start = bench_get_time_ns();
       p = alloc->realloc(p, sz);
@@ -334,7 +326,7 @@ static void wl_syn_010_run(allocator_t *alloc, bench_metrics_t *metrics) {
 
   for (size_t i = 0; i < BENCH_WARMUP_OPS; i++) {
     size_t nmemb = bench_rng_range(&rng, 1, 256);
-    size_t size = bench_rng_range(&rng, 16, 16); // 16-4096 total
+    size_t size = bench_rng_range(&rng, 16, 16);
     void *p = alloc->calloc(nmemb, size);
     alloc->free(p);
   }