On Wed, 2023-09-06 at 21:16 +0200, priour...@gmail.com wrote: [...snip...]
> Signed-off-by: benjamin priour <vultk...@gcc.gnu.org> > Co-authored-by: david malcolm <dmalc...@redhat.com> Please also add: Signed-off-by: David Malcolm <dmalc...@redhat.com> [...snip...] > > +static bool > +compatible_epath_p (const exploded_path *lhs_path, > + const exploded_path *rhs_path) > +{ > + gcc_assert (lhs_path); > + gcc_assert (rhs_path); > + gcc_assert (rhs_path->length () > 0); > + gcc_assert (rhs_path->length () > 0); > + int lhs_eedge_idx = lhs_path->length () -1; > + int rhs_eedge_idx = rhs_path->length () -1; Minor formatting nit: there should be a space between the '-' and the '1' in the above lines, hence: int lhs_eedge_idx = lhs_path->length () - 1; int rhs_eedge_idx = rhs_path->length () - 1; [...snip...] OK for trunk with those changes Thanks Dave