[patch] Disable tree-ssa DSE for now

Jeffrey A Law law@redhat.com
Wed Jan 19 06:43:00 GMT 2005


On Tue, 2005-01-18 at 21:58 -0800, Mark Mitchell wrote:
> Daniel Berlin wrote:
> > 
> > 
> > On Tue, 18 Jan 2005, Diego Novillo wrote:
> > 
> >> Steven Bosscher wrote:
> >>
> >>> Thoughts?
> >>>
> >> Well, given your analysis, disabling it is not a bad idea.  OTOH, DSE 
> >> doesn't take a long time to run, does it?  I don't think I've ever 
> >> noticed it.
> >>
> > 
> > Also, just because a pass is fast doesn't mean we should keep it enabled :)
> > Down this path, slowness (because it's unnecessary work) lies.
> > We didn't get slower 30 or 40% a time, it was .7% or .3%
> 
> I agree.  If the optimizer experts agree that this pass is ineffective, 
> Steven's patch should be applied.
It largely depends on what one considers ineffective.  I can say with
absolute confidence that the current implementation is able to eliminate
stores that the RTL implementation is unable to eliminate.




Jeff



More information about the Gcc-patches mailing list