[PATCH] D22440: Use a faster implementation of maxUIntN.

Justin Lebar via llvm-commits llvm-commits at lists.llvm.org
Sat Jul 16 23:10:00 PDT 2016


jlebar added inline comments.

================
Comment at: include/llvm/Support/MathExtras.h:323
@@ -323,1 +322,3 @@
+  // branch.
+  return ~UINT64_C(0) >> (64 - N);
 }
----------------
majnemer wrote:
> `~UINT64_C(0)` could be written as `UINT_MAX`
Oh.  That is better.

Thank you for the review!


https://reviews.llvm.org/D22440





More information about the llvm-commits mailing list