]> sourceware.org Git - glibc.git/commitdiff
conform/linknamespace: whitelist matherrf/matherrl
authorMike Frysinger <vapier@gentoo.org>
Thu, 23 Jul 2015 03:11:10 +0000 (23:11 -0400)
committerMike Frysinger <vapier@gentoo.org>
Thu, 23 Jul 2015 07:28:53 +0000 (03:28 -0400)
glibc supports the deprecated matherr hook for math error reporting.  The
conform tests take this into consideration and whitelist this symbol when
running linknamespace tests.

The ia64 libm code has long provided two additional hooks in this space:
matherrf (for floats)
matherrl (for long doubles)

Which causes the conform tests to fail with chains that all look like:
[initial] __atan2 ->
[libm.a(e_atan2.o)] __libm_error_support ->
[libm.a(libm_error.o)] matherrf

We can't (losslessly) redirect existing usage of these funcs to matherr
because the structure passed in is different -- matherr uses a struct with
doubles while matherrf/matherrl use floats and long doubles respectively.
Plus, this has been part of the exported ABI since glibc-2.2.3, so it
doesn't feel right to change it so late.

Until we get around to obsoleting matherr entirely, whitelist these two
additional ia64 symbols.

ChangeLog
conform/linknamespace.pl

index 47afb89d6a04e55ddfae86232c290fd6c67eb11d..488768ffcee6f898f08285d0267eb46526900cc9 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2015-07-23  Mike Frysinger  <vapier@gentoo.org>
+
+       * conform/linknamespace.pl (@whitelist): Add matherrf and matherrl.
+
 2015-07-23  Mike Frysinger  <vapier@gentoo.org>
 
        * sysdeps/unix/sysv/linux/ia64/bits/sigaction.h (struct sigaction):
index 847d2dd33a15be95d8e05efc3b61aebe55fce747..15fa6134ab425dac167d147adce2f9e6ca71efc2 100644 (file)
@@ -49,9 +49,11 @@ close (STDSYMS) || die ("close $stdsyms_file: $!\n");
 # * Bug 18442: re_syntax_options wrongly brought in by regcomp and
 # used by re_comp.
 #
-# * False positive: matherr only used conditionally.
+# * False positive: matherr only used conditionally.  matherrf/matherrl are used
+# by IA64 too for the same reason.
 #
-@whitelist = qw(signgam stdin stdout stderr re_syntax_options matherr);
+@whitelist = qw(signgam stdin stdout stderr re_syntax_options matherr matherrf
+               matherrl);
 foreach my $sym (@whitelist) {
   $stdsyms{$sym} = 1;
 }
This page took 0.120528 seconds and 5 git commands to generate.