* [PATCH] ARM fixed-point support [3/6]: missing FIXED_CST in tree-ssa-sccvn.c:copy_reference_ops_from_ref
@ 2011-05-13 14:47 Julian Brown
2011-06-30 13:43 ` Richard Earnshaw
0 siblings, 1 reply; 2+ messages in thread
From: Julian Brown @ 2011-05-13 14:47 UTC (permalink / raw)
To: gcc-patches
[-- Attachment #1: Type: text/plain, Size: 462 bytes --]
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.
[-- Attachment #2: fixed-point-sccvn-fixed-cst-2.diff --]
[-- Type: text/x-patch, Size: 569 bytes --]
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:
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] ARM fixed-point support [3/6]: missing FIXED_CST in tree-ssa-sccvn.c:copy_reference_ops_from_ref
2011-05-13 14:47 [PATCH] ARM fixed-point support [3/6]: missing FIXED_CST in tree-ssa-sccvn.c:copy_reference_ops_from_ref Julian Brown
@ 2011-06-30 13:43 ` Richard Earnshaw
0 siblings, 0 replies; 2+ messages in thread
From: Richard Earnshaw @ 2011-06-30 13:43 UTC (permalink / raw)
To: Julian Brown; +Cc: gcc-patches
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.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-06-30 13:23 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-05-13 14:47 [PATCH] ARM fixed-point support [3/6]: missing FIXED_CST in tree-ssa-sccvn.c:copy_reference_ops_from_ref Julian Brown
2011-06-30 13:43 ` Richard Earnshaw
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).