Fix CosSingle to be correct

dotnet · Jul 13, 2024 · ba147a8 · ba147a8
1 parent 19183c5
commit ba147a8
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/src/libraries/System.Private.CoreLib/src/System/Runtime/Intrinsics/VectorMath.cs b/src/libraries/System.Private.CoreLib/src/System/Runtime/Intrinsics/VectorMath.cs
@@ -215,26 +215,25 @@ public static TVectorSingle CosSingle<TVectorSingle, TVectorInt32, TVectorDouble
 
             TVectorSingle result;
 
-            if (TVectorSingle.LessThanOrEqualAll(ax, TVectorSingle.Create(ARG_LARGE)))
+            if (TVectorInt32.LessThanOrEqualAll(ux, TVectorInt32.Create(ARG_LARGE)))
             {
                 // We must be a finite value: (pi / 4) >= |x|
 
                 if (TVectorInt32.GreaterThanAny(ux, TVectorInt32.Create(ARG_SMALL - 1)))
                 {
                     // at least one element is: |x| >= 2^-13
-                    TVectorSingle x2 = x * x;
 
                     if (TVectorSingle.Count == TVectorDouble.Count)
                     {
                         result = Narrow<TVectorDouble, TVectorSingle>(
-                            CosSingleSmall(Widen<TVectorSingle, TVectorDouble>(x2))
+                            CosSingleSmall(Widen<TVectorSingle, TVectorDouble>(x))
                         );
                     }
                     else
                     {
                         result = Narrow<TVectorDouble, TVectorSingle>(
-                            CosSingleSmall(WidenLower<TVectorSingle, TVectorDouble>(x2)),
-                            CosSingleSmall(WidenUpper<TVectorSingle, TVectorDouble>(x2))
+                            CosSingleSmall(WidenLower<TVectorSingle, TVectorDouble>(x)),
+                            CosSingleSmall(WidenUpper<TVectorSingle, TVectorDouble>(x))
                         );
                     }
                 }