This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PR c++/21087] don't keep builtin anticipated decl, override it with actual declaration
- From: Alexandre Oliva <aoliva at redhat dot com>
- To: Mark Mitchell <mark at codesourcery dot com>
- Cc: gcc-patches at gcc dot gnu dot org, gcc-bugzilla at gcc dot gnu dot org
- Date: 22 Apr 2005 23:45:52 -0300
- Subject: Re: [PR c++/21087] don't keep builtin anticipated decl, override it with actual declaration
- Organization: Red Hat Global Engineering Services Compiler Team
- References: <oru0m2qw61.fsf@livre.redhat.lsd.ic.unicamp.br><42684C36.5010402@codesourcery.com>
On Apr 21, 2005, Mark Mitchell <mark@codesourcery.com> wrote:
> Alexandre Oliva wrote:
>> When push_overloaded_decl() was passed a new declaration that matches
>> a builtin decl, it would verify that the declarations matched and, if
>> so, leave the existing (built-in) declaration alone.
>> The intended behavior is to merge the built-in declaration with the
>> new declaration, into the location of the built-in declaration.
>> The problem is that duplicate_decl() doesn't perform such merging
>> when
>> the new declaration is a template decl, and then we end up with an
>> overload involving the template decl and the anticipated built-in
>> decl. However, overloads involving anticipated decls are something we
>> try to avoid, and actually check for elsewhere.
>> This patch fixes the code such that, if the existing decl is
>> anticipated and the two decls weren't merged, we discard the built-in
>> and use the new decl by itself.
>> Bootstrapped and regtested on amd64-linux-gnu. Ok to install?
> OK.
Ok for 4.0 branch as well? The same patch applies cleanly there, and
it's just completed bootstrap and regtesting on amd64-linux-gnu in the
branch as well.
--
Alexandre Oliva http://www.ic.unicamp.br/~oliva/
Red Hat Compiler Engineer aoliva@{redhat.com, gcc.gnu.org}
Free Software Evangelist oliva@{lsd.ic.unicamp.br, gnu.org}