https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93385

--- Comment #21 from rguenther at suse dot de <rguenther at suse dot de> ---
On Fri, 17 Apr 2020, jakub at gcc dot gnu.org wrote:

> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93385
> 
> --- Comment #20 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
> Looking at tree-ssa-dce.c, it uses remove_phi_node rather than gsi_remove for
> PHIs.  And for non-PHIs, it calls release_defs after gsi_remove.

Btw, I'd much prefer to not first copy the stmts and then remove them.
Instead the DCE "analysis" can be done on the original IL and stmts
be "marked" to be elided during copying.  That saves generating
SSA names and gimple stmts rather than needing to remove them after the
fact.

Reply via email to