| Message ID | 20110513145749.01aa1e16@rex.config |
|---|---|
| State | New |
| Headers | show |
On 13/05/11 14:57, Julian Brown wrote: > This patch adds FIXED_CST to copy_reference_ops_from_ref in > tree-ssa-sccvn.c. Some test code (using fixed-point constants) failed > for me without it. This looks obvious to me, unless there's some > particular reason fixed-point constants need to be handled differently > (or not at all) in that function. > > Tested alongside other patches in the series. OK to apply? > > Julian > > ChangeLog > > gcc/ > * tree-ssa-sccvn.c (copy_reference_ops_from_ref): Support FIXED_CST. > OK (provide that no tree-ssa maintainer objects in the next 24 hours). R.
commit 33446ea85573a6bd9ab000b2d8a6966e0e63c03e
Author: Julian Brown <julian@henry8.codesourcery.com>
Date: Fri May 13 05:46:15 2011 -0700
Add missing FIXED_CST to copy_reference_ops_from_ref.
diff --git a/gcc/tree-ssa-sccvn.c b/gcc/tree-ssa-sccvn.c
index 8f96d6c..55c1f53 100644
--- a/gcc/tree-ssa-sccvn.c
+++ b/gcc/tree-ssa-sccvn.c
@@ -681,6 +681,7 @@ copy_reference_ops_from_ref (tree ref, VEC(vn_reference_op_s, heap) **result)
case COMPLEX_CST:
case VECTOR_CST:
case REAL_CST:
+ case FIXED_CST:
case CONSTRUCTOR:
case VAR_DECL:
case PARM_DECL:
This patch adds FIXED_CST to copy_reference_ops_from_ref in tree-ssa-sccvn.c. Some test code (using fixed-point constants) failed for me without it. This looks obvious to me, unless there's some particular reason fixed-point constants need to be handled differently (or not at all) in that function. Tested alongside other patches in the series. OK to apply? Julian ChangeLog gcc/ * tree-ssa-sccvn.c (copy_reference_ops_from_ref): Support FIXED_CST.