Awesome. Thanks!<div><br></div><div>-eric<br><br>On Thursday, March 20, 2014, Juergen Ributzka <<a href="mailto:juergen@apple.com">juergen@apple.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Eric,<br>
<br>
I fully agree with you. I didn’t realize what little monster I created there over time. I decided to revert it and break it up a little, so it will be also more digestible later on if someone just sees the commits and not the review emails.<br>

<br>
Cheers,<br>
Juergen<br>
<br>
On Mar 20, 2014, at 1:29 PM, Eric Christopher <<a>echristo@gmail.com</a>> wrote:<br>
<br>
> For the record, feel free to post-commit review it per normal, it<br>
> wasn't that kind of request. Just "hey, can you make it easier to do<br>
> post-commit review on" :)<br>
><br>
> -eric<br>
><br>
> On Thu, Mar 20, 2014 at 1:24 PM, Eric Christopher <<a>echristo@gmail.com</a>> wrote:<br>
>> Really appreciate it. Thanks Juergen! :)<br>
>><br>
>> -eric<br>
>><br>
>> On Thu, Mar 20, 2014 at 1:21 PM, Juergen Ributzka <<a>juergen@apple.com</a>> wrote:<br>
>>> Ok, I will try to break it up into sensible pieces.<br>
>>><br>
>>> -Juergen<br>
>>><br>
>>> On Mar 20, 2014, at 1:13 PM, Eric Christopher <<a>echristo@gmail.com</a>> wrote:<br>
>>><br>
>>>> Actually, this patch is almost impossible to review otherwise. Would<br>
>>>> you mind doing this? :)<br>
>>>><br>
>>>> -eric<br>
>>>><br>
>>>> On Thu, Mar 20, 2014 at 1:12 PM, Eric Christopher <<a>echristo@gmail.com</a>> wrote:<br>
>>>>> Is there some reason this couldn't be split out into refactoring and<br>
>>>>> then adding new functionality?<br>
>>>>><br>
>>>>> -eric<br>
>>>>><br>
>>>>> On Thu, Mar 20, 2014 at 12:55 PM, Juergen Ributzka <<a>juergen@apple.com</a>> wrote:<br>
>>>>>> Author: ributzka<br>
>>>>>> Date: Thu Mar 20 14:55:52 2014<br>
>>>>>> New Revision: 204389<br>
>>>>>><br>
>>>>>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=204389&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=204389&view=rev</a><br>
>>>>>> Log:<br>
>>>>>> [Constant Hoisting] Extend coverage of the constant hoisting pass.<br>
>>>>>><br>
>>>>>> This commit extends the coverage of the constant hoisting pass, adds additonal<br>
>>>>>> debug output and updates the function names according to the style guide.<br>
>>>>>><br>
>>>>>> Related to <rdar://problem/16381500><br>
>>>>>><br>
>>>>>> Modified:<br>
>>>>>>   llvm/trunk/include/llvm/Analysis/TargetTransformInfo.h<br>
>>>>>>   llvm/trunk/lib/Analysis/TargetTransformInfo.cpp<br>
>>>>>>   llvm/trunk/lib/Target/X86/X86TargetTransformInfo.cpp<br>
>>>>>>   llvm/trunk/lib/Transforms/Scalar/ConstantHoisting.cpp<br>
>>>>>>   llvm/trunk/test/CodeGen/X86/lsr-interesting-step.ll<br>
>>>>>>   llvm/trunk/test/CodeGen/X86/negate-add-zero.ll<br>
>>>>>>   llvm/trunk/test/Transforms/ConstantHoisting/X86/phi.ll<br>
>>>>>><br>
>>>>>> Modified: llvm/trunk/include/llvm/Analysis/TargetTransformInfo.h<br>
>>>>>> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Analysis/TargetTransformInfo.h?rev=204389&r1=204388&r2=204389&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Analysis/TargetTransformInfo.h?rev=204389&r1=204388&r2=204389&view=diff</a><br>

>>>>>> ==============================================================================<br>
>>>>>> --- llvm/trunk/include/llvm/Analysis/TargetTransformInfo.h (original)<br>
>>>>>> +++ llvm/trunk/include/llvm/Analysis/TargetTransformInfo.h Thu Mar 20 14:55:52 2014<br>
>>>>>> @@ -297,10 +297,10 @@ public:<br>
>>>>>>  /// \brief Return the expected cost of materialization for the given integer<br>
>>>>>>  /// immediate of the specified type for a given instruction. The cost can be<br>
>>>>>>  /// zero if the immediate can be folded into the specified instruction.<br>
>>>>>> -  virtual unsigned getIntImmCost(unsigned Opcode, const APInt &Imm,<br>
>>>>>> -                 </blockquote></div>