r198815 - Used the DataLayout methods instead of the Module methods.

Rafael EspĂ­ndola rafael.espindola at gmail.com
Thu Jan 9 13:29:43 PST 2014


>> Modified: cfe/trunk/lib/CodeGen/CGObjCGNU.cpp
>> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGObjCGNU.cpp?rev=198815&r1=198814&r2=198815&view=diff
>> ==============================================================================
>> --- cfe/trunk/lib/CodeGen/CGObjCGNU.cpp (original)
>> +++ cfe/trunk/lib/CodeGen/CGObjCGNU.cpp Wed Jan  8 18:17:51 2014
>> @@ -947,8 +947,7 @@ CGObjCGNU::CGObjCGNU(CodeGenModule &cgm,
>>   Int32Ty = llvm::Type::getInt32Ty(VMContext);
>>   Int64Ty = llvm::Type::getInt64Ty(VMContext);
>>
>> -  IntPtrTy =
>> -      TheModule.getPointerSize() == llvm::Module::Pointer32 ? Int32Ty : Int64Ty;
>> +  IntPtrTy = CGM.getDataLayout().getPointerSize() == 32 ? Int32Ty : Int64Ty;
>
> Shouldn’t both this line and the line below actually call getPointerSizeInBits()?

Yes! I will fix it, but do you happen to have a testcase?

Thanks,
Rafael




More information about the cfe-commits mailing list