Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(19)

Unified Diff: src/IceTypeConverter.cpp

Issue 805943002: Remove TypeConverter and Module from minimal subzero build. (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Patch Set: Add missing comment. Created 6 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/IceTypeConverter.h ('k') | src/IceTypes.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/IceTypeConverter.cpp
diff --git a/src/IceTypeConverter.cpp b/src/IceTypeConverter.cpp
index 75e79b8d9aa6debc0b91f2c0620cf206ba2dc11f..ddb6cc73978d51836ab1e7b0316ad1e244047443 100644
--- a/src/IceTypeConverter.cpp
+++ b/src/IceTypeConverter.cpp
@@ -20,26 +20,29 @@ namespace Ice {
TypeConverter::TypeConverter(llvm::LLVMContext &Context) {
addLLVMType(IceType_void, llvm::Type::getVoidTy(Context));
- addLLVMType(IceType_i1, llvm::IntegerType::get(Context, 1));
- addLLVMType(IceType_i8, llvm::IntegerType::get(Context, 8));
- addLLVMType(IceType_i16, llvm::IntegerType::get(Context, 16));
- addLLVMType(IceType_i32, llvm::IntegerType::get(Context, 32));
+ llvm::Type *Type_i1 = llvm::IntegerType::get(Context, 1);
+ llvm::Type *Type_i8 = llvm::IntegerType::get(Context, 8);
+ llvm::Type *Type_i16 = llvm::IntegerType::get(Context, 16);
+ llvm::Type *Type_i32 = llvm::IntegerType::get(Context, 32);
+ llvm::Type *Type_f32 = llvm::Type::getFloatTy(Context);
+ addLLVMType(IceType_i1, Type_i1);
+ addLLVMType(IceType_i8, Type_i8);
+ addLLVMType(IceType_i16, Type_i16);
+ addLLVMType(IceType_i32, Type_i32);
addLLVMType(IceType_i64, llvm::IntegerType::get(Context, 64));
- addLLVMType(IceType_f32, llvm::Type::getFloatTy(Context));
+ addLLVMType(IceType_f32, Type_f32);
addLLVMType(IceType_f64, llvm::Type::getDoubleTy(Context));
- addLLVMType(IceType_v4i1, llvm::VectorType::get(LLVMTypes[IceType_i1], 4));
- addLLVMType(IceType_v8i1, llvm::VectorType::get(LLVMTypes[IceType_i1], 8));
- addLLVMType(IceType_v16i1, llvm::VectorType::get(LLVMTypes[IceType_i1], 16));
- addLLVMType(IceType_v16i8, llvm::VectorType::get(LLVMTypes[IceType_i8], 16));
- addLLVMType(IceType_v8i16, llvm::VectorType::get(LLVMTypes[IceType_i16], 8));
- addLLVMType(IceType_v4i32, llvm::VectorType::get(LLVMTypes[IceType_i32], 4));
- addLLVMType(IceType_v4f32, llvm::VectorType::get(LLVMTypes[IceType_f32], 4));
- assert(LLVMTypes.size() == static_cast<size_t>(IceType_NUM));
+ addLLVMType(IceType_v4i1, llvm::VectorType::get(Type_i1, 4));
+ addLLVMType(IceType_v8i1, llvm::VectorType::get(Type_i1, 8));
+ addLLVMType(IceType_v16i1, llvm::VectorType::get(Type_i1, 16));
+ addLLVMType(IceType_v16i8, llvm::VectorType::get(Type_i8, 16));
+ addLLVMType(IceType_v8i16, llvm::VectorType::get(Type_i16, 8));
+ addLLVMType(IceType_v4i32, llvm::VectorType::get(Type_i32, 4));
+ addLLVMType(IceType_v4f32, llvm::VectorType::get(Type_f32, 4));
+ assert(LLVM2IceMap.size() == static_cast<size_t>(IceType_NUM));
}
void TypeConverter::addLLVMType(Type Ty, llvm::Type *LLVMTy) {
- assert(static_cast<size_t>(Ty) == LLVMTypes.size());
- LLVMTypes.push_back(LLVMTy);
LLVM2IceMap[LLVMTy] = Ty;
}
@@ -47,7 +50,7 @@ Type TypeConverter::convertToIceTypeOther(llvm::Type *LLVMTy) const {
switch (LLVMTy->getTypeID()) {
case llvm::Type::PointerTyID:
case llvm::Type::FunctionTyID:
- return getIcePointerType();
+ return getPointerType();
default:
return Ice::IceType_NUM;
}
« no previous file with comments | « src/IceTypeConverter.h ('k') | src/IceTypes.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698