[PATCH]: Fix gfortran regressions
Daniel Berlin
dberlin@dberlin.org
Sun Jul 1 20:43:00 GMT 2007
I forgot that VIEW_CONVERT_EXPR was a tcc_reference expression, which
means it gets value numbered like the rest of our loads/stores.
The default case of value numbering references used to just happily
ignore the operation in question if it wasn't handled.
This means it would value number VIEW_CONVERT<int4>("A4") and
VIEW_CONVERT<int4>("A5") to the same.
To prevent this from happening in the future, I have modified the
default case to be gcc_unreachable, and made the things we were
*expecting* to fall into the default, explicit.
I will commit this once java finishes testing, it has already been
verified to fix the gfortran regressions.
Bootstrapped and regtested on i686-darwin and i686-linux
2007-07-01 Daniel Berlin <dberlin@dberlin.org>
* tree-ssa-sccvn.c (copy_reference_ops_from_ref): Handle constants
and ohter expected operations explicitly, change default to
gcc_unreachable.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: gfortran.diff
Type: text/x-patch
Size: 1236 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/fortran/attachments/20070701/d680d522/attachment.bin>
More information about the Fortran
mailing list