[PATCH] D22440: Use a faster implementation of maxUIntN.
David Majnemer via llvm-commits
llvm-commits at lists.llvm.org
Sat Jul 16 18:13:23 PDT 2016
majnemer added a subscriber: majnemer.
majnemer accepted this revision.
majnemer added a reviewer: majnemer.
majnemer added a comment.
This revision is now accepted and ready to land.
LGTM with nits.
================
Comment at: include/llvm/Support/MathExtras.h:323
@@ -323,1 +322,3 @@
+ // branch.
+ return ~UINT64_C(0) >> (64 - N);
}
----------------
`~UINT64_C(0)` could be written as `UINT_MAX`
https://reviews.llvm.org/D22440
More information about the llvm-commits
mailing list