[PATCH] TargetInfo: Use DataLayout to compute pointer width in getPointerWidth()

[Alp Toker]

On 26/06/2014 00:13, Rafael Espíndola wrote:
>> Hi Tom,
>>
>> TargetInfo is meant to be independent of DataLayout IIRC, otherwise frontend
>> functionality like semantic analysis would become dependent on backend
>> targets.
> Well, on the target datalayout. That is currently constructed in
> clang. My only concern is if this would make it harder to change clang
> in the future to use llvm provided datalayouts.

Oh, this is a clang DataLayout? If there's no backend dependency it's 
probably OK.

It's interesting no other target has done it this way before, but I 
guess if it can be tested with a -fsyntax-only test that doesn't have 
REQUIRES: r600-registered-target there's no fundamental problem

Alp.

>
> Cheers,
> Rafael

-- 
http://www.nuanti.com
the browser experts