This is an archive of the discontinued LLVM Phabricator instance.

[scudo] Use stdint types for internal types (redo)
ClosedPublic

Authored by cryptoad on Aug 16 2021, 11:59 AM.

Details

Summary

This is a redo of D108089 that broke some 32-bit builds.

scudo::uptr was defined as an unsigned long on 32-b platform,
while a uintptr_t is usually defined as an unsigned int.
This worked, this was not consistent, particularly with regard to
format string specifiers.

As suggested by Vitaly, since we are including stdint.h, define
the internal scudo integer types to those.

Diff Detail

Event Timeline

cryptoad requested review of this revision.Aug 16 2021, 11:59 AM
cryptoad created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptAug 16 2021, 11:59 AM
Herald added a subscriber: Restricted Project. · View Herald Transcript
vitalybuka accepted this revision.Aug 16 2021, 1:33 PM
This revision is now accepted and ready to land.Aug 16 2021, 1:33 PM
This revision was automatically updated to reflect the committed changes.