> On 04/27/2015 02:31 AM, Jan Hubicka wrote: > >Hi, > >build_common_builtin_nodes declares both __builtin_alloca and > >__builtin_alloca_with_align to have library name "alloca". This actually > >triggers warning in an updated ODR violation detector on "alloca" being > >declared twice. > > > >__builtin_alloca_with_align IMO do not have library equivalent and I think > >this > >is a pasto (__builtin_alloca_with_align is not documented in extend.texi). It > >is not clear to me if there was some intention behind this oddity though. > > > >I have bootstrapped/regtested x86_64 with the following. OK? > > > >Honza > > > > * tree.c (build_common_builtin_nodes): Do not build > > __builtin_alloca_with_align as equivalent of library alloca. > Yea, it's probably a pasto/thinko. OK for the trunk.
Thank you! I also looked into extend.texi and both __builtin_alloca and __builtin_alloca_with_align is missing (along with many other builtins) which is probably quite bad omission (__builtin_alloca_with_align is used at some places, i.e. in Firefox) Sandra, since you d oreat job on updating the docs recently, perhaps this could get at your radar? Honza > > jeff
