[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