you meant 2 checkins that showed diff between the stmt got dce'd or not?
Sun
On Tue, Oct 18, 2011 at 4:35 PM, 朱庆 <zqing1...@gmail.com> wrote:
> I'll double check on this, there is only one checkin between two days
> result.
>
> zhuqing
> 2011/10/18 Sun Chan <sun.c...@gmail.com>:
> > This is strange. Must be a regression from elsewhere. I am sure the
> identity
> > stmt had been removable without your change before. Is this fix the right
> > fix?
> > Fred, your comment?
> > Sun
> >
> > On Sun, Oct 16, 2011 at 6:12 PM, <s...@open64.net> wrote:
> >>
> >> Author: shivaram
> >> Date: 2011-10-16 06:12:14 -0400 (Sun, 16 Oct 2011)
> >> New Revision: 3764
> >>
> >> Modified:
> >> trunk/osprey/be/opt/opt_dce.cxx
> >> Log:
> >> Fix for bug#787. Removal of identity assignment statements(i=i) is
> >> prevented if lhs is not propagatable.
> >> CR by Ye,Mei and Gautam
> >>
> >>
> >>
> >> Modified: trunk/osprey/be/opt/opt_dce.cxx
> >> ===================================================================
> >> --- trunk/osprey/be/opt/opt_dce.cxx 2011-10-14 08:12:52 UTC (rev
> 3763)
> >> +++ trunk/osprey/be/opt/opt_dce.cxx 2011-10-16 10:12:14 UTC (rev
> 3764)
> >> @@ -2149,7 +2149,8 @@
> >>
> >> if (OPERATOR_is_scalar_store (opr) &&
> >> Enable_identity_removal() &&
> >> - stmt->Is_identity_assignment_removable()) // if COPYPROP assumes
> >> the stmt is deleted
> >> + stmt->Is_identity_assignment_removable() &&
> >> + !(stmt->Lhs()->Flags() & CF_DONT_PROP)) // if COPYPROP assumes
> the
> >> stmt is deleted
> >> return FALSE;
> >>
> >> // statements with zero-version chi nodes are required
> >> @@ -3262,7 +3263,8 @@
> >>
> >> if (OPERATOR_is_scalar_store (opr) &&
> >> Enable_identity_removal() &&
> >> - stmt->Is_identity_assignment_removable()) {
> >> + stmt->Is_identity_assignment_removable() &&
> >> + !(stmt->Lhs()->Flags() & CF_DONT_PROP)) {
> >> // process the rhs expression, if any
> >> CODEREP *rhs = stmt->Rhs();
> >> if ( rhs != NULL ) {
> >>
> >>
> >>
> >>
> ------------------------------------------------------------------------------
> >> All the data continuously generated in your IT infrastructure contains a
> >> definitive record of customers, application performance, security
> >> threats, fraudulent activity and more. Splunk takes this data and makes
> >> sense of it. Business sense. IT sense. Common sense.
> >> http://p.sf.net/sfu/splunk-d2d-oct
> >> _______________________________________________
> >> Open64-devel mailing list
> >> Open64-devel@lists.sourceforge.net
> >> https://lists.sourceforge.net/lists/listinfo/open64-devel
> >
> >
> >
> ------------------------------------------------------------------------------
> > All the data continuously generated in your IT infrastructure contains a
> > definitive record of customers, application performance, security
> > threats, fraudulent activity and more. Splunk takes this data and makes
> > sense of it. Business sense. IT sense. Common sense.
> > http://p.sf.net/sfu/splunk-d2d-oct
> > _______________________________________________
> > Open64-devel mailing list
> > Open64-devel@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/open64-devel
> >
> >
>
------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure contains a
definitive record of customers, application performance, security
threats, fraudulent activity and more. Splunk takes this data and makes
sense of it. Business sense. IT sense. Common sense.
http://p.sf.net/sfu/splunk-d2d-oct
_______________________________________________
Open64-devel mailing list
Open64-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/open64-devel
