Re: conditional lim

2015-07-15 Thread Evgeniya Maenkova
On Tue, Jul 14, 2015 at 2:54 PM, Richard Biener wrote: > On Mon, Jun 29, 2015 at 4:21 PM, Evgeniya Maenkova > wrote: >> On Mon, Jun 29, 2015 at 5:10 PM, Richard Biener >> wrote: >>> On Tue, Jun 9, 2015 at 10:11 PM, Evgeniya Maenkova >>> wrote: >>

Re: conditional lim

2015-06-29 Thread Evgeniya Maenkova
On Mon, Jun 29, 2015 at 5:10 PM, Richard Biener wrote: > On Tue, Jun 9, 2015 at 10:11 PM, Evgeniya Maenkova > wrote: >> On Tue, Jun 9, 2015 at 3:46 PM, Richard Biener >> wrote: >>> On Fri, May 29, 2015 at 3:14 PM, Evgeniya Maenkova >>> wrote: >>>>

Re: conditional lim

2015-06-09 Thread Evgeniya Maenkova
On Tue, Jun 9, 2015 at 3:46 PM, Richard Biener wrote: > On Fri, May 29, 2015 at 3:14 PM, Evgeniya Maenkova > wrote: >> Hi Richard, >> >> Here is some explanation. I hope you let me know if I need to clarify >> something. >> >> Also, you asked me abou

Re: conditional lim

2015-05-29 Thread Evgeniya Maenkova
more details. Basic blocks can have maps to identify if we created phi node for given variable in given basic_block, so we only add an edge argument for such case (phi_map = new hash_map*>). In short, that’s all. Thanks, Evgeniya On Sat, May 9, 2015 at 1:07 AM, Evgeniya Maenkova wrote: &g

Re: conditional lim

2015-05-27 Thread Evgeniya Maenkova
On Wed, May 27, 2015 at 2:11 PM, Richard Biener wrote: > On Tue, May 26, 2015 at 3:10 PM, Evgeniya Maenkova > wrote: >> Hi, Richard >> >> Thanks for review starting. >> >> Do you see any major issues with this patch (i.e. algorithms and ideas >> that

Re: conditional lim

2015-05-26 Thread Evgeniya Maenkova
PM, Evgeniya Maenkova > wrote: >> Hi, >> >> Could you please review my patch for predicated lim? >> >> Let me note some details about it: >> >> >> >> 1) Phi statements are still moved only if they have 1 or 2 >> arguments. Howev

[patch] conditional lim

2015-05-10 Thread Evgeniya Maenkova
d and other developers. Thanks, Evgeniya On Sat, May 9, 2015 at 4:41 PM, Andi Kleen wrote: > Evgeniya Maenkova writes: >> >> So, in my opinion it’s ok to generate additional phi node for debug >> case. But I’m not a compiler expert and maybe there is some >> requ

conditional lim

2015-05-08 Thread Evgeniya Maenkova
Hi, Could you please review my patch for predicated lim? Let me note some details about it: 1) Phi statements are still moved only if they have 1 or 2 arguments. However, phi statements could be move under conditions (as it’s done for the other statements). Probably, phi statement motion