[llvm-commits] [PATCH] Add getPointerTypeSizeInBits API to datalayout

Chandler Carruth chandlerc at google.com
Tue Nov 6 01:23:41 PST 2012


On Tue, Nov 6, 2012 at 1:12 AM, Duncan Sands <baldrick at free.fr> wrote:
> Hi Micah,
>
>
> On 05/11/12 22:41, Micah Villmow wrote:
>>
>> This re-adds the getPointerTypeSizeInBits API to the DataLayout class and
>> also adds in unit tests to verify the functionality for both scalar pointers
>> and vector of pointers.
>
>
> is this method actually useful?

I removed it from DataLayout during my cleanup specifically because
the code was more clear and more clean without it. getTypeSizeInBits
seems entirely sufficient for the purpose.



More information about the llvm-commits mailing list