On 02/06/2023 18:45, Jakub Jelinek wrote: > On Fri, Jun 02, 2023 at 06:18:38PM +0200, Rainer Orth wrote: > > Hi Alex, > > > > > g:7aae58b04b92303ccda3ead600be98f0d4b7f462 introduced -Wformat errors > > > breaking bootstrap on some targets. This patch fixes that. > > > > > > Committed as obvious. > > > > > > Thanks, > > > Alex > > > > > > gcc/ChangeLog: > > > > > > * btfout.cc (btf_asm_type): Use PRIu64 instead of %lu for uint64_t. > > > (btf_asm_datasec_type): Likewise. > > > > This is PR libstdc++/110077. Btw., your fix is incomplete: it needs > > another change (%lu -> %zu) in btf_asm_func_type. > > Can we rely on %zu working? Sure, it is in C99 and so in C++11 as well, > but I don't see it being used inside of gcc/ at all and not sure if all host > C libraries support it.
Looks like the follow-up patch from David fixes this without relying on %zu: https://gcc.gnu.org/pipermail/gcc-patches/2023-June/620524.html Alex > > Jakub >
