On Tue, 5 Nov 2013, Ilya Enkovich wrote:

> > So the next question is why this file includes expr.h.  It's a tree-*
> > file.  The presumption is that it should not need anything to do with RTL
> > and so shouldn't need to include expr.h, rtl.h or tm.h, directly or
> > indirectly.
> >
> > If there are just a few bits of RTL-related code for this functionality,
> > but most of it works on GIMPLE, I'd suggest separating the RTL pieces out
> > into a separate file.
> 
> It exports some functions manipulating with RTL and used by expand and
> target.  Iface macros DECL_BOUNDS_RTL and SET_DECL_BOUNDS_RTL also use
> rtx type.  Should I make two separate .h and .c files or include RTL
> part into something existent?

I'd suggest separate files.

-- 
Joseph S. Myers
jos...@codesourcery.com

Reply via email to