From 8986361fa15a4560d8a241bae726bb237b5c27e5 Mon Sep 17 00:00:00 2001 From: John McCall Date: Thu, 1 Dec 2016 23:51:30 +0000 Subject: [PATCH] Struct GEPs must use i32, not whatever size_t is. It should be safe to do this unconditionally, given that the indices will always be small constant integers anyway. llvm-svn: 288440 --- clang/lib/CodeGen/ConstantBuilder.h | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/clang/lib/CodeGen/ConstantBuilder.h b/clang/lib/CodeGen/ConstantBuilder.h index 6c5a5e5..40b34a9 100644 --- a/clang/lib/CodeGen/ConstantBuilder.h +++ b/clang/lib/CodeGen/ConstantBuilder.h @@ -248,11 +248,13 @@ public: // Otherwise, add an index to drill into the first level of pointer. } else { assert(indices.empty()); - indices.push_back(llvm::ConstantInt::get(Builder.CGM.SizeTy, 0)); + indices.push_back(llvm::ConstantInt::get(Builder.CGM.Int32Ty, 0)); } assert(position >= Begin); - indices.push_back(llvm::ConstantInt::get(Builder.CGM.SizeTy, + // We have to use i32 here because struct GEPs demand i32 indices. + // It's rather unlikely to matter in practice. + indices.push_back(llvm::ConstantInt::get(Builder.CGM.Int32Ty, position - Begin)); } }; -- 2.7.4