This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Fix PR47691: always run scev_const_prop before graphite

On Tue, 26 Jul 2011, Sebastian Pop wrote:

> On Tue, Jul 26, 2011 at 09:33, Richard Guenther <rguenther@suse.de> wrote:
> > On Tue, 26 Jul 2011, Sebastian Pop wrote:
> >
> >> Ping.
> >> Any opinions on this patch?
> >
> > Well, I don't think we should do this. ?Why does the user disable
> > scev-const-prop when enabling graphite? ?If he does so, fine - he
> > has to live with worse codegen.
> 
> Here is the patch.  Ok for trunk after regstrap?

Required?  Do you mean you generate wrong code in graphite if it is
not run?  What if it just fails to handle things or does not handle
them because things are too expensive?

Please make graphite more robust instead.

Thanks,
Richard.
Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]