<div dir="ltr">Sorry for the delay. Only comment is that the formatting would have been nice to look at as a separate patch since it made it hard to see the rest :)<div><br></div><div>Thank you very much for doing this!</div>
<div><br></div><div>-eric</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Aug 22, 2014 at 7:44 PM, Juergen Ributzka <span dir="ltr"><<a href="mailto:juergen@apple.com" target="_blank">juergen@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Finally got around to do some spring cleaning :P I guess the function names still deserve some bikeshedding.<br>
<br>
Cheers,<br>
Juergen<br>
<br>
<br><br>
<br>
<br>
On Aug 14, 2014, at 11:02 AM, Jim Grosbach <<a href="mailto:grosbach@apple.com">grosbach@apple.com</a>> wrote:<br>
<br>
> Indeed. Just remember, you touch it you own it. ;)<br>
><br>
> -Jim<br>
><br>
>> On Aug 14, 2014, at 10:52 AM, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br>
>><br>
>> Thanks for all the work Juergen!<br>
>><br>
>> -eric<br>
>><br>
>> On Thu, Aug 14, 2014 at 10:52 AM, Juergen Ributzka <<a href="mailto:juergen@apple.com">juergen@apple.com</a>> wrote:<br>
>>> Sure, will do.<br>
>>><br>
>>> On Aug 14, 2014, at 10:50 AM, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br>
>>><br>
>>>> Excellent. Let's do it.<br>
>>>><br>
>>>> On Thu, Aug 14, 2014 at 10:49 AM, Jim Grosbach <<a href="mailto:grosbach@apple.com">grosbach@apple.com</a>> wrote:<br>
>>>>> When there’s active and steady development in the area already is when it makes sense to go ahead and do style and conventions cleanups like this. Go for it.<br>
>>>>><br>
>>>>> -Jim<br>
>>>>><br>
>>>>>> On Aug 14, 2014, at 10:43 AM, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br>
>>>>>><br>
>>>>>> Mmm.. that way may lie madness. I'm always a proponent of big changes<br>
>>>>>> to make things consistent, but let's use a 3rd party to check.<br>
>>>>>><br>
>>>>>> Jim? :)<br>
>>>>>><br>
>>>>>> -eric<br>
>>>>>><br>
>>>>>> On Thu, Aug 14, 2014 at 10:41 AM, Juergen Ributzka <<a href="mailto:juergen@apple.com">juergen@apple.com</a>> wrote:<br>
>>>>>>> Should we also change tblgen to not emit capitalized function names anymore for FastEmit_*?<br>
>>>>>>><br>
>>>>>>> -Juergen<br>
>>>>>>><br>
>>>>>>> On Aug 14, 2014, at 10:38 AM, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br>
>>>>>>><br>
>>>>>>>> That would be pretty nice. Generally we don't do the mass changes like<br>
>>>>>>>> that, but I think the file is pretty awkward right now and so I'd<br>
>>>>>>>> appreciate it for sure :)<br>
>>>>>>>><br>
>>>>>>>> -eric<br>
>>>>>>>><br>
>>>>>>>> On Thu, Aug 14, 2014 at 10:37 AM, Juergen Ributzka <<a href="mailto:juergen@apple.com">juergen@apple.com</a>> wrote:<br>
>>>>>>>>> Copy paste from TargetMaterializeConstant ;) The file is a little bit of a<br>
>>>>>>>>> mess when it comes to function names. Should I do a cleanup and update all<br>
>>>>>>>>> the function names while I am at it?<br>
>>>>>>>>><br>
>>>>>>>>> -Juergen<br>
>>>>>>>>><br>
>>>>>>>>><br>
>>>>>>>>> On Aug 13, 2014, at 6:30 PM, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br>
>>>>>>>>><br>
>>>>>>>>>><br>
>>>>>>>>>><br>
>>>>>>>>>> -/// materializeRegForValue - Helper for getRegForValue. This function is<br>
>>>>>>>>>> -/// called when the value isn't already available in a register and must<br>
>>>>>>>>>> -/// be materialized with new instructions.<br>
>>>>>>>>>> -unsigned FastISel::materializeRegForValue(const Value *V, MVT VT) {<br>
>>>>>>>>>> +unsigned FastISel::MaterializeConstant(const Value *V, MVT VT) {<br>
>>>>>>>>>> unsigned Reg = 0;<br>
>>>>>>>>>> -<br>
>>>>>>>>><br>
>>>>>>>>><br>
>>>>>>>>> Coding standard has functions with lower case names? :)<br>
>>>>>>>>><br>
>>>>>>>>> -eric<br>
>>>>>>>>><br>
>>>>>>>>><br>
>>>>>>><br>
>>>>><br>
>>><br>
><br>
<br>
<br></blockquote></div><br></div>