This is an archive of the discontinued LLVM Phabricator instance.

[VE] Change inetger constants 32-bit friendly
ClosedPublic

Authored by kaz7 on Jan 29 2021, 8:01 PM.

Details

Summary

Correct integer constants like 1UL << 63 to UINT64_C(1) << 63 in
order to make them work on 32-bit machines. Tested on both an i386
and x86_64 machines.

Diff Detail

Event Timeline

kaz7 created this revision.Jan 29 2021, 8:01 PM
kaz7 requested review of this revision.Jan 29 2021, 8:01 PM
mgorny accepted this revision.Feb 1 2021, 1:32 AM
mgorny added a subscriber: mgorny.

I can confirm that the tests pass for me with this patch.

This revision is now accepted and ready to land.Feb 1 2021, 1:32 AM
This revision was landed with ongoing or failed builds.Feb 1 2021, 2:01 AM
This revision was automatically updated to reflect the committed changes.