[PATCH, PR 42585] Total scalarization of small records

Steve Ellcey sje@cup.hp.com
Fri Jan 22 18:25:00 GMT 2010


On Fri, 2010-01-22 at 19:05 +0100, Jakub Jelinek wrote:
> On Fri, Jan 22, 2010 at 09:59:16AM -0800, Steve Ellcey wrote:
> > This patch breaks platforms that don't define their own MOVE_RATIO,
> > Since I haven't seen a patch to fix the problem, does someone want to
> > approve this patch to fix it?  Only MOVE_RATIO has to be in defaults.h
> > but I moved CLEAR_RATIO and SET_RATIO too in order to keep them
> > together.
> > 
> > I successully built this on ia64-hp-hpux11.23 and am currently running
> > the test suite.
> 
> I don't see how this can work.
> tm.h first includes defaults.h, and only after it it includes insn-flags.h.
> So I believe none of the HAVE_* macros will be defined.
> 
> IMHO you instead want to include expr.h in whatever file now needs it.
> 
> 	Jakub


That sounds good,  I am rebuilding with this patch:

2010-01-22  Steve Ellcey  <sje@cup.hp.com>

        * tree-sra.c: Add include of expr.h.


Index: tree-sra.c
===================================================================
--- tree-sra.c  (revision 156167)
+++ tree-sra.c  (working copy)
@@ -77,6 +77,7 @@ along with GCC; see the file COPYING3.  
 #include "alloc-pool.h"
 #include "tm.h"
 #include "tree.h"
+#include "expr.h"
 #include "gimple.h"
 #include "cgraph.h"
 #include "tree-flow.h"





More information about the Gcc-patches mailing list