[Bug tree-optimization/65533] [5 Regression] 252.eon in SPEC CPU 2000 failed to build

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65533

--- Comment #5 from Richard Biener <rguenth at gcc dot gnu.org> ---
(In reply to Jakub Jelinek from comment #4)
> Created attachment 35120 [details]
> gcc5-pr65533.patch
> 
> Untested fix.

Hmm, looks basically ok, though doing

  vect_free_slp_tree (child);

from the cleanup before

      /* If the SLP build for operand zero failed and operand zero
         and one can be commutated try that for the scalar stmts
         that failed the match.  */
      if (i == 0
...

and simply re-allocating child with

      child = vect_create_new_slp_node (oprnd_info->def_stmts);
      if (!child)
        {
          vect_free_oprnd_info (oprnds_info);
          return false;
        }

might be "simpler".

Your patch is ok if it is already tested.