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: Fix double_int overflow in VRP PLUS_EXPR

From: Marc Glisse <marc dot glisse at inria dot fr>
To: Jakub Jelinek <jakub at redhat dot com>
Cc: Richard Guenther <richard dot guenther at gmail dot com>, gcc-patches at gcc dot gnu dot org
Date: Wed, 22 Aug 2012 14:34:01 +0200 (CEST)
Subject: Re: Fix double_int overflow in VRP PLUS_EXPR
References: <alpine.DEB.2.02.1208221346210.21614@stedding.saclay.inria.fr> <CAFiYyc1pB1XhidVvepjWzqsux=8YUimsLc_RsbWyu_jDmXNgAA@mail.gmail.com> <20120822122246.GS1999@tucnak.redhat.com>

On Wed, 22 Aug 2012, Jakub Jelinek wrote:

2012-08-21  Marc Glisse  <marc.glisse@inria.fr>
+  n <<= (8 * sizeof (NT) - 1);
Better use __CHAR_BIT__ instead of 8 here...

Ok, I committed the __CHAR_BIT__ version (I just compiled that one file manually with old and new compilers, I didn't restart the testsuite, I hope the manual test is enough to detect any typo that could break things).

--
Marc Glisse

Follow-Ups:
- Re: Fix double_int overflow in VRP PLUS_EXPR
  - From: Jakub Jelinek

References:
- Fix double_int overflow in VRP PLUS_EXPR
  - From: Marc Glisse
- Re: Fix double_int overflow in VRP PLUS_EXPR
  - From: Richard Guenther
- Re: Fix double_int overflow in VRP PLUS_EXPR
  - From: Jakub Jelinek

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