https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90883
Jim Wilson <wilson at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |wilson at gcc dot gnu.org
--- Comment #29 from Jim Wilson <wilson at gcc dot gnu.org> ---
The testcase works for riscv64-elf but does not work for riscv32-elf. The
difference is in the einline pass before dse1. riscv64-elf has
tmp.C:12:17: optimized: Inlining constexpr C::C()/1 into C slow()/3.
where as riscv32-elf has
tmp.C:12:17: missed: will not early inline: C slow()/3->constexpr
C::C()/1, call is cold and code would grow by 1
Since the constructor was not early inlined, dse1 can't eliminate the redundant
store. The constructor eventually gets inlined between
085i.materialize-all-clones and 088t.fixup_cfg3 which allows dse2 to eliminate
the redundant store.
I can make the testcase work for riscv32-elf if I add
--param max-inline-insns-size=1
to allow the constructor to be inlined during the einline pass. I didn't check
to see if this works for the other failing targets.
