On Sun, 10 Jul 2011, Joern Rennecke wrote:

> Quoting "Joseph S. Myers" <jos...@codesourcery.com>:
> 
> > On Tue, 5 Jul 2011, Joern Rennecke wrote:
> > 
> > > This patch splits out a new generator genattr-enum from genattr, and it
> > > generates insn-attr-enum.h, which just makes the enum declarations.
> > > This new header file is then included by options.c and insn-attr.h .
> > 
> > Is there a particular reason for making this separate from the existing
> > genattr-common that I created recently?  Like opts.c, options.c is a file
> > shared by both the driver and the core compiler that can't include the
> > full insn-attr.h for the same reason.
> 
> Attached is a variant of my patch that uses your suggestion.
> bootstrapped on x86_64-unknown-linux-gnu.

OK in the absence of objections within 48 hours.

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

Reply via email to