[llvm-commits] [PATCH] Additional SD-Node Cleanup after Combining Pass

Hal Finkel hfinkel at anl.gov
Sun Apr 8 13:08:34 PDT 2012


On Sun, 08 Apr 2012 19:57:15 +0200
Duncan Sands <baldrick at free.fr> wrote:

> Hi Hal,
> 
> > Ping. (This is a one-line change, and I'd like to get this fixed for
> > Gabor).
> 
> isn't it a bug that the DAG combiner isn't deleting dead nodes?

It is certainly a bug that *something* is not deleting the dead
nodes. Are you suggesting that the call to
RemoveDeadNodes() should actually come from Combine()? or that there is
some deeper structural problem?

Thanks again,
Hal

> 
> Ciao, Duncan.
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits



-- 
Hal Finkel
Postdoctoral Appointee
Leadership Computing Facility
Argonne National Laboratory



More information about the llvm-commits mailing list