This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch] Rename tree-flow.h to tree-cfg.h.


On 10/17/2013 08:48 PM, Andrew MacLeod wrote:

rs6000 and alpha were including tree-flow.h in their config/target.c file. I built stage 1 for each target as a cross compiler to confirm builds dont break due to something unresolved. rs6000.c no longer needed it, alpha needs num_ssa_names, so it requires gimple-ssa.h (instead of tree-flow.h). There is a compilation error in alpha (like on many other targets) for target_flags_explicit.... when it fails, that is the only unresolved external left.... the situation which existed before this patch.

Bootstraps on x86_64-unknown-linux-gnu with no new regressions.

OK?

Andrew

Apparently rs6000.c requires cgraph when HAVE_AS_GNU_ATTRIBUTE is defined, which it isn't for the cross-compile I tried using target rs6000-ibm-aix4.3. I also tried powerpc64-unknown-linux-gnu, but it still doesn't fail. I hacked up the file to define the macro and verify that simply including cgraph.h is sufficient to compile the file.

Thanks to jbglaw and his build robot for snagging this so quickly for me.

Checked in as revision 203824.

Andrew




2013-10-18  Andrew MacLeod  <amacleod@redhat.com>

	* config/rs6000/rs6000.c: Include cgraph.h.

Index: config/rs6000/rs6000.c
===================================================================
*** config/rs6000/rs6000.c	(revision 203817)
--- config/rs6000/rs6000.c	(working copy)
***************
*** 58,63 ****
--- 58,64 ----
  #include "opts.h"
  #include "tree-vectorizer.h"
  #include "dumpfile.h"
+ #include "cgraph.h"
  #if TARGET_XCOFF
  #include "xcoffout.h"  /* get declarations of xcoff_*_section_name */
  #endif

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]