[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