[PATCH] D18258: Have DataLayout::isLegalInteger accept a uint64_t

[Mehdi AMINI via llvm-commits]

joker.eph accepted this revision.
joker.eph added a comment.
This revision is now accepted and ready to land.

LGTM with a nit below.


================
Comment at: include/llvm/IR/DataLayout.h:239
@@ -238,3 +238,3 @@
   /// The width is specified in bits.
-  bool isLegalInteger(unsigned Width) const {
+  bool isLegalInteger(uint64_t Width) const {
     for (unsigned LegalIntWidth : LegalIntWidths)
----------------
I guess the case where `Width > MAX_INT_BITS` is rare enough that it's not worth an early exit here?

================
Comment at: test/Transforms/SROA/pr26972.ll:10
@@ +9,3 @@
+  %t0 = bitcast [1073741825 x i32]* %a to i8*
+  call void @llvm.lifetime.end(i64 4294967300, i8* %t0)
+  ret void
----------------
Add a comment specifying that this constant is intended to trigger 32 bits overflow.


http://reviews.llvm.org/D18258