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