On Jan  4, 2021, Richard Biener <richard.guent...@gmail.com> wrote:

> Hmm - while the change looks good, doesn't it end up
> calling end_imm_use_stmt_tranverse twice for those
> uses still calling BREAK_FROM_IMM_USE_STMT?
It does.  I'd considered introducing a separate method to call
end_imm_use_stmt_traverse if imm is not NULL, and then set it to NULL,
but calling the function multiple times is not a problem: delink_imm_use
just returns immediately the second time.

> Thus, please remove uses of BREAK_FROM_IMM_USE_STMT
> together with this patch.

And RETURN_FROM_IMM_USE_STMT, I suppose?

I wasn't sure whether to remove them and their users.

-- 
Alexandre Oliva, happy hacker  https://FSFLA.org/blogs/lxo/
   Free Software Activist         GNU Toolchain Engineer
        Vim, Vi, Voltei pro Emacs -- GNUlius Caesar

Reply via email to