On Tue, 2021-03-23 at 12:48 +0000, Richard Sandiford wrote: > Ilya Leoshkevich <i...@linux.ibm.com> writes: > > +inline use_info * > > +set_info::single_nondebug_use () const > > +{ > > + use_info *nondebug_insn = single_nondebug_insn_use (); > > + if (nondebug_insn) > > + return has_phi_uses () ? nullptr : nondebug_insn; > > + use_info *phi = single_phi_use (); > > + if (phi) > > + return has_nondebug_insn_uses() ? nullptr : phi; > > + return nullptr; > > Very minor, but I think this is simpler as: > > if (!has_phi_uses ()) > return single_nondebug_insn_use (); > if (!has_nondebug_insn_uses ()) > return single_phi_use (); > return nullptr; > > OK with that change (or without if you prefer the original). > Thanks for the fix and for your patience. :-) > > Richard
Retested with the change above and pushed as: https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=b61461ac7f9bdd0e98145be79423d19b933afaa0 Thanks for all the suggestions! Best regards, Ilya