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]
Other format:	[Raw text]

Re: Fwd: [PATCH] Fix C++/39365 ++ operator with volatile bool increments

From: Andrew Pinski <pinskia at gmail dot com>
To: Jason Merrill <jason at redhat dot com>
Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Thu, 17 Sep 2009 16:02:02 -0700
Subject: Re: Fwd: [PATCH] Fix C++/39365 ++ operator with volatile bool increments
References: <de8d50360904161040i7d91172bmb10f35dacae79a6e@mail.gmail.com> <de8d50360906211551u66121938n9b2056ad3fcd210a@mail.gmail.com> <4A5AF9AF.3050006@redhat.com>

On Mon, Jul 13, 2009 at 2:09 AM, Jason Merrill <jason@redhat.com> wrote:
> On 06/22/2009 12:51 AM, Andrew Pinski wrote:
>>
>> * typeck.c (cp_build_unary_op): Check TREE_CODE for bools instead of
>> using same_type_p.
>
> OK, but please make the same change to type_promotes_to and
> convert_for_assignment.

Here is the final patch which I applied after another bootstrap/test
on i386-darwin8.11.

Thanks,
Andrew Pinski

cp/ChangeLog:
* typeck.c (cp_build_unary_op): Check TREE_CODE for bools instead of
using same_type_p.
(convert_for_assignment): Likewise.
* cvt.c (type_promotes_to): Likewise.

testsuite/ChangeLog:
* g++.dg/expr/bool3.C: New test.
* g++.dg/expr/bool4.C: New test.

Attachment: fixbool.diff.txt
Description: Text document

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