http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37516

--- Comment #5 from rguenther at suse dot de <rguenther at suse dot de> 
2012-02-24 08:38:19 UTC ---
On Thu, 23 Feb 2012, pinskia at gcc dot gnu.org wrote:

> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=37516
> 
> --- Comment #4 from Andrew Pinski <pinskia at gcc dot gnu.org> 2012-02-23 
> 22:15:05 UTC ---
> 
> > Maybe get away with these old-stylish names ('tree' and 'fold') and
> > make it match reality, gimple-ssa-combine.c ;)
> 
> That sounds like a good idea.  I have done that.
> 
> > 
> > Still have to have a look at your branch - do you have a brief
> > overview documentation, for example on the wiki?  What I'd like
> > to do is have an interface that I can re-use for VN combining,
> > basically replace its simplify_*_expression functions.
> 
> I don't have one yet.  But I hope to create one this weekend.
> 
> Right now replacing the VN combining with this new functionality will not work
> as it scans the defs itself.  Though I am moving all the def scanning to its
> own function which should allow for the code to be more modular and allow for
> the VN combining to work with it.

I think def scanning is ok, but all SSA names should be "valueized" via
a hook first.

Richard.

Reply via email to