Patchwork ARM fixed-point support [3/6]: missing FIXED_CST in tree-ssa-sccvn.c:copy_reference_ops_from_ref

login
register
mail settings
Submitter Julian Brown
Date May 13, 2011, 1:57 p.m.
Message ID <20110513145749.01aa1e16@rex.config>
Download mbox | patch
Permalink /patch/95480/
State New
Headers show

Comments

Julian Brown - May 13, 2011, 1:57 p.m.
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.
Richard Earnshaw - June 30, 2011, 1:23 p.m.
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.

Patch

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: