Re: RFA: Making attribute values avaliable for options

2011-07-21 Thread Joseph S. Myers
On Sun, 10 Jul 2011, Joern Rennecke wrote: > Quoting "Joseph S. Myers" : > > > 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 head

Re: RFA: Making attribute values avaliable for options

2011-07-10 Thread Joern Rennecke
Quoting "Joseph S. Myers" : 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 pa

Re: RFA: Making attribute values avaliable for options

2011-07-05 Thread Joseph S. Myers
On Tue, 5 Jul 2011, Joern Rennecke wrote: > > 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. > > Well, FWIW, at the moment, I can actually build both cc1 and xgcc when > I hand-edit options.c to

Re: RFA: Making attribute values avaliable for options

2011-07-05 Thread Joern Rennecke
Quoting "Joseph S. Myers" : 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 pa

Re: RFA: Making attribute values avaliable for options

2011-07-05 Thread Joseph S. Myers
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

RFA: Making attribute values avaliable for options

2011-07-05 Thread Joern Rennecke
There is often an enum corresponding to a target.md attribute that you want as numeric values for an Enum option declaration. Alas, insn-attr.h is not included by options.c, and an attempt to include it with the HeaderInclude record is doomed because of all the headers that insn-attr.h requires t