[Bug fortran/45159] Unneccessary temporaries

tkoenig at gcc dot gnu dot org gcc-bugzilla@gcc.gnu.org
Sun Aug 1 18:12:00 GMT 2010



------- Comment #4 from tkoenig at gcc dot gnu dot org  2010-08-01 18:12 -------
This piece of code

                 /* If no intention of reversing or reversing is explicitly
                     inhibited, convert backward dependence to overlap.  */
                  if ((reverse == NULL && this_dep == GFC_DEP_BACKWARD)
                        || (reverse && reverse[n] == GFC_CANNOT_REVERSE))
                    this_dep = GFC_DEP_OVERLAP;

looks more fishy.  It sets this_dep to GFC_DEP_OVERLAP even if we don't want to
reverse at all.

Shouldn't this all be conditional on

if (this_dep == GFC_DEP_BACKWARD)

Paul, do you have any ideas?


-- 

tkoenig at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pault at gcc dot gnu dot org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45159



More information about the Gcc-bugs mailing list