This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Fix for PR c++/59031
- From: Easwaran Raman <eraman at google dot com>
- To: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Cc: Jason Merrill <jason at redhat dot com>, Paolo Carlini <paolo dot carlini at oracle dot com>
- Date: Fri, 15 Nov 2013 09:52:47 -0800
- Subject: Re: Fix for PR c++/59031
- Authentication-results: sourceware.org; auth=none
- References: <CAPK5YPatQxsM+awi=4YUdjgfQ0UmG+K4zNhxct=r2QZXF5BohA at mail dot gmail dot com> <CAPK5YPbYx41fwOTXRuYvpAuSyPVYMfgvFVMgce79iGpSJ2C8fw at mail dot gmail dot com>
Ping.
On Mon, Nov 11, 2013 at 9:46 AM, Easwaran Raman <eraman@google.com> wrote:
> Ping.
>
> On Thu, Nov 7, 2013 at 11:14 AM, Easwaran Raman <eraman@google.com> wrote:
>> Before r193504, if a method can not be overridden, LOOKUP_NONVIRTUAL
>> is set and the call is direct. The changes at r193504 (to fix PR
>> c++/11750) caused a regression to this behavior. This patch attempts
>> to fix that. Bootstraps and no test regressions on x86_64/linux. Is
>> this a correct fix and is this ok for trunk?
>>
>> Thanks,
>> Easwaran
>>
>> 2013-11-07 Easwaran Raman <eraman@google.com>
>>
>> PR c++/59031
>> * call.c (build_new_method_call_1): Comnpare function context
>> with BASELINK_BINFO type rather than instance type before
>> marking the call with LOOKUP_NONVIRTUAL.
>>
>> testsuite/ChangeLog:
>>
>> 2013-11-07 Easwaran Raman <eraman@google.com>
>>
>> PR c++/59031
>> * g++.dg/inherit/virtual11.C: New test.