[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