On Wed, May 22, 2019 at 09:39:52AM +0200, Martin Liška wrote:
> The patch is about using of uintptr_t instead unsigned long that's
> being used for printing value of a pointer.
> 
> Patch can bootstrap on x86_64-linux-gnu and survives regression tests.
> 
> Ready to be installed?

No.  You haven't adjusted the fprintf format strings for this change,
so it will break build on any target where uintptr_t is not unsigned long.

I don't see any gcc/ code using the PRIxPTR and similar macros, so no idea
about the portability of that, but I'd think we just should keep that as is.

        Jakub

Reply via email to