Re: [PATCH] libstdc++-v3: testsuite: Prune uncapitalized "in function" linker warning
Jacob Bachmeyer <[email protected]> Wed, 14 Aug 2024 20:58:04 -0500
| Newsgroups | gmane.comp.sysutils.dejagnu.general,gmane.comp.gcc.patches,gmane.comp.gcc.libstdc++.devel |
|---|---|
| Message-ID | <[email protected]> |
Hans-Peter Nilsson wrote: > (CC to the dejagnu project as a heads-up) > > Regtested cris-elf with a fresh newlib checkout where 2640 > libstdc++-v3 tests otherwise fail due to the stubbed newlib > _getentropy. Ok to commit? > > -- >8 -- > Newer newlib trigger warnings about certain functions not implemented > (_getentropy) when testing libstdc++-v3. > > Since 2018 (circa binutils-2.10) the "in function" prefix isn't > capitalized for those "not implemented" warnings when generated from > the linker (a GNU ld feature used by newlib). Dejagnu up to and > including at least dejagnu-1.6.3 (and git @ 42979bd3b9) assumes a > capital "In function", leaving that part unpruned, and boom we have > thousands of "excess errors" from the libstdc++-v3 testsuite. > To confirm: newer binutils drops capitalization in a message "in function FOO\nBAR is not implemented and will always fail". DejaGnu expects GNU ld to say "In function FOO..." and fails to recognize the new form. If this is correct, I will fix it in Git master. (The fix is trivial: change the "I" to "[Ii]" to accept both forms.) (You may still want to add the patch to the testsuite, for compatibility with older versions of DejaGnu.) -- Jacob