llvm dev - Oct 2009 - [LLVMdev] PR3707

Hello,

I'm back :) This patch fixes pr3707.

Regards
-Jakub
-------------- next part --------------
A non-text attachment was scrubbed...
Name: pr3707.patch
Type: application/octet-stream
Size: 3247 bytes
Desc: not available
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20091010/301a2761/attachment.obj>

On Oct 10, 2009, at 7:48 AM, Jakub Staszak wrote:
> Hello,
>
> I'm back :)
Great!
> This patch fixes pr3707.
Can you explain a little more what this does?  What is the intuition  
behind disabling this optimization?

-Chris

On Oct 12, 2009, at 2:27 AM, Chris Lattner wrote:
> On Oct 10, 2009, at 7:48 AM, Jakub Staszak wrote:
>
>> This patch fixes pr3707.
>
> Can you explain a little more what this does?  What is the intuition  
> behind disabling this optimization?
Oh, this patch is crap :( Sorry for this.

The problem is that we lack of optimization for case like:

loopto:
   %timeout.0 = phi i32 [ 2000, %loopto ], [ %timeout.0.ph,  
%loopto.outer ]

Then something like "%reg<def> = MOV 2000" is inserted into
loopto
block and executed every time. I haven't found any optimization which  
can handle this and move instruction into some outer block.

-- 
Jakub Staszak