<div dir="ltr">Glibc 2.39 branch also has this issue.<div><br></div><div>OK to backport?</div></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Fri, Dec 6, 2024 at 4:59 PM Sunil Pandey <<a href="mailto:skpgkp2@gmail.com">skpgkp2@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Dec 5, 2024 at 2:42 AM H.J. Lu <<a href="mailto:hjl.tools@gmail.com" target="_blank">hjl.tools@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Thu, Dec 5, 2024 at 6:29 PM Joseph Myers <<a href="mailto:josmyers@redhat.com" target="_blank">josmyers@redhat.com</a>> wrote:<br>
><br>
> On Thu, 5 Dec 2024, H.J. Lu wrote:<br>
><br>
> > On Thu, Dec 5, 2024 at 9:04 AM H.J. Lu <<a href="mailto:hjl.tools@gmail.com" target="_blank">hjl.tools@gmail.com</a>> wrote:<br>
> > ><br>
> > > Since internal tests don't have access to internal symbols in libm,<br>
> > > exclude them for internal tests.  Also make tst-strtod5 and tst-strtod5i<br>
> > > depend on $(libm) to support older versions of GCC which can't inline<br>
> > > copysign family functions.  This fixes BZ #32414.<br>
> > ><br>
> > ><br>
> > > --<br>
> > > H.J.<br>
> ><br>
> > It is used in glibc build for some targets:<br>
> ><br>
> > FAIL: glibcs-powerpc-linux-gnu build<br>
> > FAIL: glibcs-powerpc-linux-gnu-power4 build<br>
> > FAIL: glibcs-powerpc64le-linux-gnu build<br>
> > FAIL: glibcs-powerpc-linux-gnu-soft build<br>
> > FAIL: glibcs-powerpc64-linux-gnu build<br>
> > FAIL: glibcs-powerpc64le-linux-gnu-disable-multi-arch build<br>
> > FAIL: glibcs-sparcv9-linux-gnu build<br>
> > FAIL: glibcs-sparc64-linux-gnu build<br>
> ><br>
> > But they shouldn't be declared in the installed header.<br>
><br>
> Please give more details of the failures you see (with this patch, as<br>
> opposed to the "[PATCH] math: Don't declare __XXX math functions [BZ<br>
> #32418]" one).  It seems rather odd for a change adding !IS_IN<br>
> (testsuite_internal) conditionals to break the build of glibc itself.<br>
> (And this patch is only changing non-installed files, it's nothing to do<br>
> with what's in installed headers.)<br>
><br>
<br>
I followed it up with the wrong patch.   This patch seems OK.<br>
<br>
<br>
-- <br>
H.J.<br></blockquote><div><br></div><div>LGTM</div><div>Reviewed-by: Sunil K Pandey <<a href="mailto:skpgkp2@gmail.com" target="_blank">skpgkp2@gmail.com</a>></div><div><br></div><div>Thanks,</div><div>Sunil </div></div></div>
</blockquote></div>