On 4/19/23 19:05, Andrew Pinski via Gcc-patches wrote:
This small patch just changes around the code slightly to make it easier to understand that the cases were handling diamond shaped BB for both do_store_elim/do_hoist_loads. There is no effect on code output at all since all of the checks are the same still. Note this depends on https://gcc.gnu.org/pipermail/gcc-patches/2023-April/616241.html as that allows this to be done. OK? Bootstrapped and tested on x86_64-linux-gnu with no regressions. gcc/ChangeLog: * tree-ssa-phiopt.cc (tree_ssa_phiopt_worker): Change the code around slightly to move diamond handling for do_store_elim/do_hoist_loads out of the big if/else.
OK. jeff
