[committed 0/2] CRIS: Fix compilation warnings that recent gcc treats as errors
Hans-Peter Nilsson
hp@axis.com
Fri Dec 15 04:24:50 GMT 2023
> Date: Wed, 6 Dec 2023 20:52:48 +0100
> From: Torbjorn SVENSSON <torbjorn.svensson@foss.st.com>
(Sorry for the late reply: the newlib mailer appears to have
directed your mail only to the newlib without even a CC to
me, so your reply got into my newlib folder, unfortunately
not visited daily.)
> Hi Hans-Peter,
>
> On 2023-12-06 18:50, Hans-Peter Nilsson wrote:
> > Caveat: a patch-set adding a _getentropy stub
> > from August (IIRC) causes lots of FAILs; some of those are worked
> > around in gcc to get better coverage.
>
> What problems are there with the _getentropy stubs that I've submitted?
I hope to get into details later, as indicated by the "film
at 11". The problem is fairly visible with a standard
test-run for cris-elf (with simulator and baseboard
cris-sim): all libstdc++ tests fail with a linker warning,
as its configure tests detect a presence of _getentropy but
its reference trigs the stub warning (the .gnu.warning
thing). I *think* it's also visible for a build with
arm-eabi+arm-sim which made me wonder how you configured and
tested (you may have stated, I haven't looked). Adding a
patch to "prune" the stub-warning in libstdc++ prune.exp
only exposes runtime errors when _getentropy is actually
called. Those errors are not present when _getentropy is
not detected (and not used).
Looks like I managed to actually state the gist of the
problem. Missing are details such as (todo) proposed best
actions, besides reverting that patch, to better handle the
problem *you* say. That's also all I can say for now.
Feel very free to wait until I come back to this: I can't
focus on this issue right now; just thought I'd reply with
what state I have at the top of my head, since you asked.
Sorry!
brgds, H-P
More information about the Newlib
mailing list