This is an archive of the discontinued LLVM Phabricator instance.

Add missing header for uint64_t
ClosedPublic

Authored by pmatos on May 22 2023, 7:32 AM.

Details

Summary

LLVM fails to build in some environments with EXPENSIVE_CHECKS
due to the use of uint64_t without including cstdint.

Diff Detail

Event Timeline

pmatos created this revision.May 22 2023, 7:32 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 22 2023, 7:32 AM
Herald added subscribers: asb, wingo. · View Herald Transcript
pmatos requested review of this revision.May 22 2023, 7:32 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 22 2023, 7:32 AM

Adding Jay and Nikita as reviewers since they authored and approved the related revision D120123 .
I have noticed that in some environment with EXPENSIVE_CHECKS, LLVM fails to build due to undefined uint64_t used in line 234.

This revision is now accepted and ready to land.May 22 2023, 7:38 AM
This revision was landed with ongoing or failed builds.May 22 2023, 7:40 AM
This revision was automatically updated to reflect the committed changes.