diff options
author | Bruno Haible <bruno@clisp.org> | 2021-01-03 11:19:22 +0100 |
---|---|---|
committer | Bruno Haible <bruno@clisp.org> | 2021-01-03 11:19:22 +0100 |
commit | 5d1dafd8346fda4e268bcf1747dc97925d1ab576 (patch) | |
tree | f9dcaf38b1e1cdf88d1dde09dd6e580142a49216 /lib/free.c | |
parent | f22259966f1f52efbf5f03d274298a6417b4595c (diff) | |
download | gnulib-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.c | 14 |
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 } |