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

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P1
             Status|UNCONFIRMED                 |ASSIGNED
   Last reconfirmed|                            |2017-01-27
           Assignee|unassigned at gcc dot gnu.org      |rguenth at gcc dot 
gnu.org
   Target Milestone|---                         |7.0
     Ever confirmed|0                           |1

--- Comment #2 from Richard Biener <rguenth at gcc dot gnu.org> ---
Probably needs enhancement here:

  /* Apply our simple cost model - fuse partitions with similar
     memory accesses.  */
  for (i = 0; partitions.iterate (i, &into); ++i)
    {
      bool changed = false;
      if (partition_builtin_p (into))
        continue;
      for (int j = i + 1;
           partitions.iterate (j, &partition); ++j)
        {
          if (!partition_builtin_p (partition)
              && similar_memory_accesses (rdg, into, partition))
            {
              if (dump_file && (dump_flags & TDF_DETAILS))
                {
                  fprintf (dump_file, "fusing partitions\n");
                  dump_bitmap (dump_file, into->stmts);

see how we do not let detected patterns participate in the cost modeling.

Reply via email to