[llvm] r215588 - [FastISel] Let the target decide first if it wants to materialize a constant.

[Eric Christopher]

That would be pretty nice. Generally we don't do the mass changes like
that, but I think the file is pretty awkward right now and so I'd
appreciate it for sure :)

-eric

On Thu, Aug 14, 2014 at 10:37 AM, Juergen Ributzka <juergen at apple.com> wrote:
> Copy paste from TargetMaterializeConstant ;) The file is a little bit of a
> mess when it comes to function names. Should I do a cleanup and update all
> the function names while I am at it?
>
> -Juergen
>
>
> On Aug 13, 2014, at 6:30 PM, Eric Christopher <echristo at gmail.com> wrote:
>
>>
>>
>> -/// materializeRegForValue - Helper for getRegForValue. This function is
>> -/// called when the value isn't already available in a register and must
>> -/// be materialized with new instructions.
>> -unsigned FastISel::materializeRegForValue(const Value *V, MVT VT) {
>> +unsigned FastISel::MaterializeConstant(const Value *V, MVT VT) {
>>    unsigned Reg = 0;
>> -
>
>
> Coding standard has functions with lower case names? :)
>
>  -eric
>
>