[llvm-commits] Cost Table, take 3

[Chandler Carruth]

On Jan 24, 2013 2:09 PM, "Renato Golin" <renato.golin at linaro.org> wrote:

> Now, in include/llvm/Target
>

Oh, please go ahead and submit. =] LGTM. I meant the prior message to
indicate you should submit with the header file move (which ever one seemed
to work best).


>
> 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/0fe8efa9/attachment.html>