Re: [PATCH] Add infrastructure to merge standard builtin enums with backend builtins

2011-08-27 Thread Richard Guenther
On Sat, Aug 27, 2011 at 12:12 AM, Mike Stump wrote: > On Aug 26, 2011, at 7:19 AM, Michael Meissner wrote: The alternative is something like what Kenney and Mike are doing in their private port, where they have new syntax in the MD file for builtins. >>> >>> But are those user-exposed bu

Re: [PATCH] Add infrastructure to merge standard builtin enums with backend builtins

2011-08-26 Thread Mike Stump
On Aug 26, 2011, at 7:19 AM, Michael Meissner wrote: >>> The alternative is something like what Kenney and Mike are doing in their >>> private port, where they have new syntax in the MD file for builtins. >> >> But are those user-exposed builtins? Certainly interesting to combine >> builtin defin

Re: [PATCH] Add infrastructure to merge standard builtin enums with backend builtins

2011-08-26 Thread Michael Meissner
On Fri, Aug 26, 2011 at 10:19:24AM +0200, Richard Guenther wrote: > On Thu, Aug 25, 2011 at 10:35 PM, Michael Meissner > wrote: > > On Wed, Aug 24, 2011 at 11:06:55AM +0200, Richard Guenther wrote: > >> This basically would make DECL_BUILT_IN_CLASS no longer necessary > >> if all targets where con

Re: [PATCH] Add infrastructure to merge standard builtin enums with backend builtins

2011-08-26 Thread Richard Guenther
On Thu, Aug 25, 2011 at 10:35 PM, Michael Meissner wrote: > On Wed, Aug 24, 2011 at 11:06:55AM +0200, Richard Guenther wrote: >> This basically would make DECL_BUILT_IN_CLASS no longer necessary >> if all targets where converted, right?  (We don't currently have any >> BUILT_IN_FRONTEND builtins).

Re: [PATCH] Add infrastructure to merge standard builtin enums with backend builtins

2011-08-25 Thread Mike Stump
On Aug 25, 2011, at 1:35 PM, Michael Meissner wrote: > The alternative is something like what Kenney and Mike are doing in their > private port, where they have new syntax in the MD file for builtins. I think the issue is actually largely orthogonal. In our code, we generate which code is used b

Re: [PATCH] Add infrastructure to merge standard builtin enums with backend builtins

2011-08-25 Thread Michael Meissner
On Wed, Aug 24, 2011 at 11:06:55AM +0200, Richard Guenther wrote: > This basically would make DECL_BUILT_IN_CLASS no longer necessary > if all targets where converted, right? (We don't currently have any > BUILT_IN_FRONTEND builtins). That would sound appealing if this > patch weren't a partial t

Re: [PATCH] Add infrastructure to merge standard builtin enums with backend builtins

2011-08-24 Thread Richard Guenther
On Tue, Aug 23, 2011 at 7:29 PM, Michael Meissner wrote: > Over the years, it has been a problem for ports like the PowerPC that have > more builtins than the standard list.  Code that references the built_in_decls > and implicit_built_in_decls arrays are supposed to check DECL_BUILT_IN_CLASS > be

Re: [PATCH] Add infrastructure to merge standard builtin enums with backend builtins

2011-08-23 Thread Paolo Bonzini
On 08/23/2011 07:29 PM, Michael Meissner wrote: + tm_builtin_funcs="$(tm_builtin_funcs) config/rs6000/rs6000-builtin.def" + tm_builtin_funcs="$(tm_builtin_funcs) config/rs6000/rs6000-builtin.def" These should use braces; build parts are otherwise okay. Paolo

Re: [PATCH] Add infrastructure to merge standard builtin enums with backend builtins

2011-08-23 Thread Mike Stump
On Aug 23, 2011, at 10:29 AM, Michael Meissner wrote: > This patch growes the bit field that has the builtin function number by one > bit. I _use_ 13 bits on my port. If you do 13, we won't have to change it for 10 years, I promise. :-)