Change compiler on windows test

.github/workflows/cmake-multi-platform.yml

@@ -17,19 +17,19 @@ jobs:
       fail-fast: false
 
       # Set up a matrix to run the following 3 configurations:
-      # 1. <Windows, Release, latest MSVC compiler toolchain on the default runner image, default generator>
+      # 1. <Windows, Release, latest GCC compiler toolchain on the default runner image, default generator>
       # 2. <Linux, Release, latest GCC compiler toolchain on the default runner image, default generator>
       # 3. <Linux, Release, latest Clang compiler toolchain on the default runner image, default generator>
       #
       # To add more build types (Release, Debug, RelWithDebInfo, etc.) customize the build_type list.
       matrix:
         os: [ubuntu-latest, windows-latest]
         build_type: [Release]
-        c_compiler: [gcc, clang, cl]
+        c_compiler: [gcc, clang]
         include:
           - os: windows-latest
-            c_compiler: cl
-            cpp_compiler: cl
+            c_compiler: gcc
+            cpp_compiler: g++
           - os: ubuntu-latest
             c_compiler: gcc
             cpp_compiler: g++

tinyqr.h

@@ -59,13 +59,12 @@ inline __attribute__((always_inline)) scalar_t inv_sqrt(scalar_t x) {
 }
 // fast inverse square root - might buy us a tiny bit, and I have been looking
 // for forever to use this :)
-/*
 template <>
 [[maybe_unused]] inline __attribute__((always_inline)) float inv_sqrt(float x) {
-  long i = *reinterpret_cast<long *>(&x);       // evil floating point bit level
-hacking // NOLINT [runtime/int] i = 0x5f3759df - (i >> 1);  // what the fuck?
+  long i = *reinterpret_cast<long *>(&x); // NOLINT [runtime/int]
+  i = 0x5f3759df - (i >> 1);
   return *reinterpret_cast<float *>(&i);
-}*/
+}
 template <typename scalar_t>
 inline __attribute__((always_inline)) std::tuple<scalar_t, scalar_t>
 givens_rotation(const scalar_t a, const scalar_t b) {