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 8/8] cfgloop.h includes basic-block.h

From: Jeff Law <law at redhat dot com>
To: Andrew MacLeod <amacleod at redhat dot com>, gcc-patches <gcc-patches at gcc dot gnu dot org>, Richard Biener <richard dot guenther at gmail dot com>
Date: Fri, 18 Oct 2013 10:06:53 -0600
Subject: Re: [patch 8/8] cfgloop.h includes basic-block.h
Authentication-results: sourceware.org; auth=none
References: <5260856C dot 7070008 at redhat dot com> <52613B96 dot 2080509 at redhat dot com>

On 10/18/13 07:45, Andrew MacLeod wrote:

Another basoc-block inclusion.  cfgloop.h was including basic-block.h,
which meant lots of other things were getting it from here too.

The only routine in cfgloop.h which uses anything from basic-block.h was
bb_loop_depth().  By moving that to cfgloop.c, its no longer required by
the .h file.    I did have to include function.h for a few routines.  I
cannot imagine this being hot enough to show up on any performance
measure...

bootstraps on x86_64-unknown-linux-gnu with no new regressions. OK?

OK.
jeff

Follow-Ups:
- Re: [patch 8/8] cfgloop.h includes basic-block.h
  - From: Andrew MacLeod

References:
- [patch] Rename tree-flow.h to tree-cfg.h.
  - From: Andrew MacLeod
- [patch 8/8] cfgloop.h includes basic-block.h
  - From: Andrew MacLeod

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