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

Drea Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|ASSIGNED                    |RESOLVED

--- Comment #4 from Drea Pinski <pinskia at gcc dot gnu.org> ---
(In reply to Richard Biener from comment #2)
> Just to mention in more places - I'd like to have a gimple_insert_iterator
> that carries before/after, GSI_{NEW_STMT,CONTINUE_LINKING,...} in addition
> to the gsi to de-bloat code inserting stmts.  It should also be able to
> insert on edges.

Makes sense.

A few places (not just in phiopt) are inserting at the begining of the Basic
block too. And that should be de-bloated too. This case here we do:
gsi = gsi_after_labels (bb); (or )
gsi_insert_before (&gsi, new_stmt, GSI_NEW_STMT/GSI_SAME_STMT);

So having a short cut for that would be nice too.

I will file a seperate bug about those 2 things.

Reply via email to