Hi Mei, Your new patch looks good to me.
Thanks, Min On Mon, Aug 22, 2011 at 12:16 PM, Ye, Mei <mei...@amd.com> wrote: > Hi Min > > Thanks a lot for your review. Attached patch combines two parts together. > W.r.t to why the last check of Postdominates is needed, consider a Fortran > code like below: > > // NODE 1 > do l=1,dim > if (yf(l).gt.cpa(l)) then > k=k+1 > endif > enddo > > // NODE 2 > if (k.eq.0) go to 120 > k = l; > return; > 120 continue; > k = j; > > NODE2 contains unstructured flows that are grouped together into one SC_BLOCK > node, the block containing "120 continue" does not post-dominates the first > block containing "if(k.eq.0) go to 120". It is illegal to hoist NODE 2 > above NODE 1. "SC_NODE::Is_sese" is used as a legality check, so it will be > desirable for it to return FALSE for a case like this. > ------------------------------------------------------------------------------ uberSVN's rich system and user administration capabilities and model configuration take the hassle out of deploying and managing Subversion and the tools developers use with it. Learn more about uberSVN and get a free download at: http://p.sf.net/sfu/wandisco-dev2dev _______________________________________________ Open64-devel mailing list Open64-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/open64-devel
