This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]

Re: [C++ PATCH] Fix g++.other/init16.C (take 2)

To: Jakub Jelinek <jakub at redhat dot com>
Subject: Re: [C++ PATCH] Fix g++.other/init16.C (take 2)
From: Jason Merrill <jason at redhat dot com>
Date: 16 Feb 2001 21:20:20 +0000
Cc: mark at codesourcery dot com, gcc-patches at gcc dot gnu dot org
References: <20010216165927.O768@sunsite.ms.mff.cuni.cz><u9r90yshhf.fsf@casey.cambridge.redhat.com><20010216122009.F16592@devserv.devel.redhat.com>

>>>>> "Jakub" == Jakub Jelinek <jakub@redhat.com> writes:

> On Fri, Feb 16, 2001 at 04:46:52PM +0000, Jason Merrill wrote:
>> I think it makes more sense to check lvalue_p.

> I tried it initially (unfortunately with !real_lvalue_p, not !lvalue_p) and
> it broke dtor10.C and dtor11.C, but this seems to work very well (passed
> make check-g++ and as I've bootstrapped successfully with both the previous
> patch or !real_lvalue_p test, I think it would bootstrap as well).
> Ok to commit (what about branch)?

It fixes a regression from 2.95, so yes to both.

Jason

References:
- [C++ PATCH] Fix g++.other/init16.C
  - From: Jakub Jelinek
- Re: [C++ PATCH] Fix g++.other/init16.C
  - From: Jason Merrill
- [C++ PATCH] Fix g++.other/init16.C (take 2)
  - From: Jakub Jelinek

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]