[llvm-commits] Cost Table, take 3

[Renato Golin]

Now, in include/llvm/Target

cheers,
--renato


On 24 January 2013 21:53, Renato Golin <renato.golin at linaro.org> wrote:

> On 24 January 2013 21:06, Chandler Carruth <chandlerc at google.com> wrote:
>
>> This code (and your plan) makes perfect sense to me.
>>
>> I would suggest putting it in its own header, or in some other header
>> under include/llvm/Target --- the consumers of the analysis aren't the ones
>> that want it.
>>
>
> I looked for a better header on Target, but there was none, and since it
> was an exclusive TransformInfo issue, I thought creating a new header on
> Target would be too much. But it makes sense, even if a really small
> header, won't be on the wrong place.
>
> Will change.
>
> cheers,
> --renato
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130124/dfeb8d47/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cost-table.patch
Type: application/octet-stream
Size: 7654 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130124/dfeb8d47/attachment.obj>