[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