From 6c2bc93874ff10b63a0e0b832dfa5340f0abad73 Mon Sep 17 00:00:00 2001 From: Aryadev Chavali Date: Thu, 27 Nov 2025 01:12:35 +0000 Subject: [PATCH] remove index_t definition conflicts with prev code base. Also std::optional doubles the size of the underlying type. horrifying! I don't want to have to give 16 bytes over for something that could be embedded in the 8 bytes of the u64 directly. I'm thinking we could just use i64's instead, sacrificing that top bit to indicate if the value is present or not. --- src/base.hpp | 7 ------- 1 file changed, 7 deletions(-) diff --git a/src/base.hpp b/src/base.hpp index 8947f32..4544661 100644 --- a/src/base.hpp +++ b/src/base.hpp @@ -30,13 +30,6 @@ static_assert(sizeof(double) == 8, "f64 requires 8 byte doubles"); using f32 = float; using f64 = double; -using index_t = i64; - -inline bool index_has_value(index_t i) -{ - return i >= 0; -} - inline u64 gcd(u64 a, u64 b) { if (a == b)