On 5/22/19 9:50 AM, Jakub Jelinek wrote:
> 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.

Ok.

Martin

> 
>       Jakub
> 

Reply via email to