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