This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix PR56466
- From: Marek Polacek <polacek at redhat dot com>
- To: Miles Bader <miles at gnu dot org>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Richard Biener <rguenther at suse dot de>
- Date: Thu, 28 Feb 2013 10:49:21 +0100
- Subject: Re: [PATCH] Fix PR56466
- References: <20130227144759.GB15445@redhat.com> <878v69j965.fsf@catnip.gol.com>
On Thu, Feb 28, 2013 at 02:32:02PM +0900, Miles Bader wrote:
> Marek Polacek <polacek@redhat.com> writes:
> > + bool changed = false;
> > + changed |= true;
> > + changed |= true;
> > + changed |= true;
> > + changed |= true;
> > + if (changed)
>
> Why do you use "|=" ...? Isn't it equivalent to just "=" (which is
> more clear) for a boolean?
Ah, yeah, it's a remnant; I had "changed = false;" there as well.
Will adjust, thanks,
Marek