]> granicus.if.org Git - graphviz/commitdiff
Revert "work around a bug in GCC: It generates a call to "sqrtf" for "(float)sqrt...
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Tue, 15 Feb 2022 08:12:52 +0000 (19:12 +1100)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Wed, 16 Feb 2022 10:08:05 +0000 (21:08 +1100)
This reverts commit 0dfd6ab3449455907a5a684fb54cb8b54c5befd5.

As of fe3f9411d2c59b463ab1b64eecfd19f2db55d2fc, Graphviz requires a C99
compiler. C99 guarantees `sqrtf` so even if the “bug” discussed in the original
commit still exists, generating a `sqrtf` call is fine on a C99 toolchain.

lib/neatogen/matrix_ops.c

index 0ad5357b298d229da99833244fba3bba740ca5cd..85d8ab370b7e288995405a3a3aafd2303f1dec83 100644 (file)
@@ -611,11 +611,8 @@ void invert_vec(int n, float *vec)
 void sqrt_vec(int n, float *vec)
 {
     int i;
-    double d;
     for (i = 0; i < n; i++) {
-       /* do this in two steps to avoid a bug in gcc-4.00 on AIX */
-       d = sqrt(vec[i]);
-       vec[i] = (float) d;
+       vec[i] = (float) sqrt(vec[i]);
     }
 }