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
