From 37b7ba22cb0087cc881b0cb55d6c0cd053c6edc5 Mon Sep 17 00:00:00 2001
From: Griffin Bassman <griffinbassman@gmail.com>
Date: Mon, 3 Jun 2024 13:19:06 -0400
Subject: [PATCH] more ci defs

---
 .github/workflows/wasm.yml             | 2 +-
 .scripts/linux/build-static-java.sh    | 2 +-
 .scripts/linux/build-with-coverage.sh  | 2 +-
 .scripts/linux/build.sh                | 2 +-
 vowpalwabbit/core/src/reductions/gd.cc | 1 -
 5 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/.github/workflows/wasm.yml b/.github/workflows/wasm.yml
index 6561733e607..b1dbd4f3ae8 100644
--- a/.github/workflows/wasm.yml
+++ b/.github/workflows/wasm.yml
@@ -26,7 +26,7 @@ jobs:
         with:
             node-version: 16
       - name: Configure
-        run: emcmake cmake --preset wasm -G Ninja -DCMAKE_BUILD_TYPE=MinSizeRel -DCMAKE_TOOLCHAIN_FILE=$(pwd)/ext_libs/vcpkg/scripts/buildsystems/vcpkg.cmake
+        run: emcmake cmake --preset wasm -G Ninja -DCMAKE_BUILD_TYPE=MinSizeRel -DSTD_INV_SQRT=ON -DCMAKE_TOOLCHAIN_FILE=$(pwd)/ext_libs/vcpkg/scripts/buildsystems/vcpkg.cmake
       - name: Build
         run: cmake --build build --target vw-wasm
       - uses: actions/upload-artifact@v3
diff --git a/.scripts/linux/build-static-java.sh b/.scripts/linux/build-static-java.sh
index 21d334cb8fe..826685fb8a5 100755
--- a/.scripts/linux/build-static-java.sh
+++ b/.scripts/linux/build-static-java.sh
@@ -12,6 +12,6 @@ cd build
 # /usr/local/bin/gcc + g++ is 9.2.0 version
 cmake -E env LDFLAGS="-Wl,--exclude-libs,ALL -static-libgcc -static-libstdc++" cmake .. -DCMAKE_BUILD_TYPE=Release -DWARNINGS=Off -DBUILD_JAVA=On -DBUILD_DOCS=Off -DVW_FEAT_FLATBUFFERS=On -DVW_FEAT_CSV=On -DVW_FEAT_CB_GRAPH_FEEDBACK=On\
  -DBUILD_PYTHON=Off -DSTATIC_LINK_VW_JAVA=On -DCMAKE_C_COMPILER=/usr/local/bin/gcc -DCMAKE_CXX_COMPILER=/usr/local/bin/g++ \
- -DBUILD_TESTING=Off -DVW_ZLIB_SYS_DEP=Off -DBUILD_SHARED_LIBS=Off -DVW_BUILD_LAS_WITH_SIMD=Off
+ -DBUILD_TESTING=Off -DVW_ZLIB_SYS_DEP=Off -DBUILD_SHARED_LIBS=Off -DVW_BUILD_LAS_WITH_SIMD=Off -DSTD_INV_SQRT=ON
 NUM_PROCESSORS=$(nproc)
 make vw_jni -j ${NUM_PROCESSORS}
diff --git a/.scripts/linux/build-with-coverage.sh b/.scripts/linux/build-with-coverage.sh
index bac8db23bac..83b4acc49d1 100755
--- a/.scripts/linux/build-with-coverage.sh
+++ b/.scripts/linux/build-with-coverage.sh
@@ -6,5 +6,5 @@ SCRIPT_DIR="$(cd "$(dirname "$0")" && pwd)"
 REPO_DIR=$SCRIPT_DIR/../../
 cd $REPO_DIR
 
-cmake -S . -B build -G Ninja -DCMAKE_BUILD_TYPE=Debug -DVW_GCOV=ON -DWARNINGS=OFF -DBUILD_JAVA=Off -DBUILD_PYTHON=Off -DBUILD_TESTING=On -DVW_FEAT_FLATBUFFERS=On -DVW_FEAT_CSV=On -DVW_FEAT_CB_GRAPH_FEEDBACK=On
+cmake -S . -B build -G Ninja -DCMAKE_BUILD_TYPE=Debug -DVW_GCOV=ON -DWARNINGS=OFF -DBUILD_JAVA=Off -DBUILD_PYTHON=Off -DBUILD_TESTING=On -DVW_FEAT_FLATBUFFERS=On -DVW_FEAT_CSV=On -DVW_FEAT_CB_GRAPH_FEEDBACK=On -DSTD_INV_SQRT=ON
 cmake --build build
diff --git a/.scripts/linux/build.sh b/.scripts/linux/build.sh
index ab886b0a349..50a61fe5d7b 100755
--- a/.scripts/linux/build.sh
+++ b/.scripts/linux/build.sh
@@ -9,5 +9,5 @@ cd $REPO_DIR
 # If parameter 1 is not supplied, it defaults to Release
 BUILD_CONFIGURATION=${1:-Release}
 
-cmake -S . -B build -G Ninja -DCMAKE_BUILD_TYPE=${BUILD_CONFIGURATION} -DWARNINGS=Off -DWARNING_AS_ERROR=On -DVW_BUILD_VW_C_WRAPPER=Off -DBUILD_JAVA=On -DBUILD_PYTHON=Off -DBUILD_TESTING=On -DBUILD_EXPERIMENTAL_BINDING=On -DVW_FEAT_FLATBUFFERS=On -DVW_FEAT_CSV=On -DVW_FEAT_CB_GRAPH_FEEDBACK=On
+cmake -S . -B build -G Ninja -DCMAKE_BUILD_TYPE=${BUILD_CONFIGURATION} -DWARNINGS=Off -DWARNING_AS_ERROR=On -DVW_BUILD_VW_C_WRAPPER=Off -DBUILD_JAVA=On -DBUILD_PYTHON=Off -DBUILD_TESTING=On -DBUILD_EXPERIMENTAL_BINDING=On -DVW_FEAT_FLATBUFFERS=On -DVW_FEAT_CSV=On -DVW_FEAT_CB_GRAPH_FEEDBACK=On -DSTD_INV_SQRT=ON
 cmake --build build --target all
diff --git a/vowpalwabbit/core/src/reductions/gd.cc b/vowpalwabbit/core/src/reductions/gd.cc
index a97c6521984..852448606b7 100644
--- a/vowpalwabbit/core/src/reductions/gd.cc
+++ b/vowpalwabbit/core/src/reductions/gd.cc
@@ -153,7 +153,6 @@ VW_WARNING_STATE_POP
 
 static inline float inv_sqrt(float x)
 {
-  return 1.f / std::sqrt(x);
 #if defined(STD_INV_SQRT)
   return 1.f / std::sqrt(x);
 #endif