Merge latest main into work

Author: bugparty

CMakeLists.txt

@@ -2,8 +2,8 @@ cmake_minimum_required(VERSION 3.21)
 project(bitvector)
 
 set(CMAKE_CXX_STANDARD 17)
-option(BV_BOUNDS_CHECK "Enable bounds checking in bitvector" ON)
-if(NOT BV_BOUNDS_CHECK)
+option(BITVECTOR_NO_BOUND_CHECK "Disable bounds checking in bitvector" OFF)
+if(BITVECTOR_NO_BOUND_CHECK)
   add_compile_definitions(BITVECTOR_NO_BOUND_CHECK)
 endif()
 if(MSVC)
@@ -34,6 +34,7 @@ elseif(MSVC)
     message(WARNING "BMI1 support is not available for MSVC in this configuration.")
 endif()
 
+# Optionally disable bounds checking in the bitvector implementation
 
 # Enable testing
 enable_testing()

README.md

@@ -7,7 +7,7 @@ This repository provides a small bit vector implementation along with tests and
 Bounds checking is enabled by default. To benchmark without checks, configure and build with (this defines `BITVECTOR_NO_BOUND_CHECK`):
 
 ```bash
-cmake -S . -B build -DBV_BOUNDS_CHECK=OFF -DCMAKE_BUILD_TYPE=Release
+cmake -S . -B build -DBITVECTOR_NO_BOUND_CHECK=ON -DCMAKE_BUILD_TYPE=Release
 cmake --build build --config Release
 ./build/bitvector_benchmark
 ```