From 1c5b1db20031026ff7e38bc08b4273323da2cc52 Mon Sep 17 00:00:00 2001
From: yutianyu <SqCBeFObcpwfC3SvwsFmWI+yutianyu@noreply.cnb.cool>
Date: Mon, 4 May 2026 19:16:49 +0800
Subject: [PATCH] refactor: consolidate benchmark entrypoint

Co-authored-by: wawahejun <hejunlbbc@gmail.com>
---
 tests/bench_all.py | 88 ----------------------------------------------
 tests/run_ops.py   |  6 ++++
 2 files changed, 6 insertions(+), 88 deletions(-)
 delete mode 100644 tests/bench_all.py

diff --git a/tests/bench_all.py b/tests/bench_all.py
deleted file mode 100644
index c5141a1..0000000
--- a/tests/bench_all.py
+++ /dev/null
@@ -1,88 +0,0 @@
-from __future__ import annotations
-
-import argparse
-import sys
-from pathlib import Path
-
-ROOT = Path(__file__).resolve().parents[1]
-PYTHON_DIR = ROOT / "python"
-if str(PYTHON_DIR) not in sys.path:
-    sys.path.insert(0, str(PYTHON_DIR))
-if str(ROOT) not in sys.path:
-    sys.path.insert(0, str(ROOT))
-
-import torch
-
-from operator_runtime.testing import PerformanceResult
-from tests.bench.copy import bench_copy
-from tests.bench.reduce_sum import bench_reduce_sum
-from tests.bench.softmax import bench_softmax
-from tests.bench.vector_add import bench_vector_add
-
-
-def _format_table(rows: list[PerformanceResult]) -> str:
-    headers = [
-        "operator",
-        "backend",
-        "shape",
-        "dtype",
-        "runtime_ms",
-        "torch_ms",
-        "speedup",
-        "GB/s",
-        "GFLOP/s",
-    ]
-    body = []
-    for row in rows:
-        speedup = "-" if row.speedup is None else f"{row.speedup:.2f}"
-        torch_ms = "-" if row.torch_ms is None else f"{row.torch_ms:.4f}"
-        body.append(
-            [
-                row.operator,
-                row.backend,
-                row.shape,
-                row.dtype,
-                f"{row.runtime_ms:.4f}",
-                torch_ms,
-                speedup,
-                f"{row.gbytes_per_sec:.2f}",
-                f"{row.gflops_per_sec:.2f}",
-            ]
-        )
-
-    widths = []
-    for idx, header in enumerate(headers):
-        content_width = max((len(r[idx]) for r in body), default=0)
-        widths.append(max(len(header), content_width))
-
-    def fmt_row(cols: list[str]) -> str:
-        return " | ".join(col.ljust(widths[idx]) for idx, col in enumerate(cols))
-
-    separator = "-+-".join("-" * width for width in widths)
-    lines = [fmt_row(headers), separator]
-    lines.extend(fmt_row(cols) for cols in body)
-    return "\n".join(lines)
-
-
-def main() -> int:
-    parser = argparse.ArgumentParser()
-    parser.add_argument("--backend", default="nvidia")
-    parser.add_argument("--profile", default=None)
-    args = parser.parse_args()
-
-    if not torch.cuda.is_available():
-        print("CUDA is required for benchmark", file=sys.stderr)
-        return 2
-
-    rows: list[PerformanceResult] = []
-    rows.extend(bench_copy(args.backend))
-    rows.extend(bench_vector_add(args.backend))
-    rows.extend(bench_reduce_sum(args.backend))
-    rows.extend(bench_softmax(args.backend))
-
-    print(_format_table(rows))
-    return 0
-
-
-if __name__ == "__main__":
-    raise SystemExit(main())
diff --git a/tests/run_ops.py b/tests/run_ops.py
index 8cb323a..62369b6 100644
--- a/tests/run_ops.py
+++ b/tests/run_ops.py
@@ -5,6 +5,8 @@ import subprocess
 import sys
 from pathlib import Path
 
+import torch
+
 ROOT = Path(__file__).resolve().parents[1]
 PYTHON_DIR = ROOT / "python"
 if str(PYTHON_DIR) not in sys.path:
@@ -116,6 +118,10 @@ def main() -> int:
     parser.add_argument("--mode", choices=["test", "bench", "all"], default="all")
     args = parser.parse_args()
 
+    if args.mode in ("bench", "all") and not torch.cuda.is_available():
+        print("CUDA is required for benchmark", file=sys.stderr)
+        return 2
+
     selected_ops = ops if args.op == "all" else (args.op,)
     rows: list[list[str]] = []
     bench_rows = []