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