summaryrefslogtreecommitdiff
path: root/lib/free.c
diff options
context:
space:
mode:
authorBruno Haible <bruno@clisp.org>2021-01-03 11:19:22 +0100
committerBruno Haible <bruno@clisp.org>2021-01-03 11:19:22 +0100
commit5d1dafd8346fda4e268bcf1747dc97925d1ab576 (patch)
treef9dcaf38b1e1cdf88d1dde09dd6e580142a49216 /lib/free.c
parentf22259966f1f52efbf5f03d274298a6417b4595c (diff)
downloadgnulib-5d1dafd8346fda4e268bcf1747dc97925d1ab576.tar.gz
free-posix: Work around GCC mis-optimization bug.
Code by Bernhard Voelker <mail@bernhard-voelker.de>. * lib/free.c (rpl_free): Add alternative complicated code for GCC.
Diffstat (limited to 'lib/free.c')
-rw-r--r--lib/free.c14
1 files changed, 14 insertions, 0 deletions
diff --git a/lib/free.c b/lib/free.c
index 135c3eb16b..5c89787aba 100644
--- a/lib/free.c
+++ b/lib/free.c
@@ -27,7 +27,21 @@ void
rpl_free (void *p)
#undef free
{
+#if defined __GNUC__ && !defined __clang__
+ /* An invalid GCC optimization
+ <https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98396>
+ would optimize away the assignments in the code below, when link-time
+ optimization (LTO) is enabled. Make the code more complicated, so that
+ GCC does not grok how to optimize it. */
+ int err[2];
+ err[0] = errno;
+ err[1] = errno;
+ errno = 0;
+ free (p);
+ errno = err[errno == 0];
+#else
int err = errno;
free (p);
errno = err;
+#endif
}