On Thu, May 13, 2021 at 9:43 AM Jeff Law via Gcc-patches <gcc-patches@gcc.gnu.org> wrote: > > > On 5/13/2021 3:06 AM, Martin Liška wrote: > > Addresses the following clang warning: > > gcc/tree-ssa-dom.c:652:33: warning: private field 'm_simplifier' is > > not used [-Wunused-private-field] > > > > Ready for master? > > Thanks > > > > gcc/ChangeLog: > > > > * tree-ssa-dom.c: Remove m_simplifier. > > I wonder if Aldy's refactor accidentally dropped the ephemeral > simplifications based on the threading path equivalences which is what I > think this is supposed to be supporting. Given that the refactor didn't > cause any regressions, if that capability did get dropped, it couldn't > be too important anymore.
The dom_opt_dom_walker is instantiated with a threader, which contains a simplifier. So it got shuffled around, but the functionality should still be there. Thanks for catching this Martin. Aldy