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: [C/C++ PATCH] Fix -Wshift-overflow with sign bit

From: Jeff Law <law at redhat dot com>
To: Marek Polacek <polacek at redhat dot com>
Cc: Toon Moene <toon at moene dot org>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Jason Merrill <jason at redhat dot com>, Joseph Myers <joseph at codesourcery dot com>
Date: Wed, 12 Aug 2015 11:46:53 -0600
Subject: Re: [C/C++ PATCH] Fix -Wshift-overflow with sign bit
Authentication-results: sourceware.org; auth=none
References: <20150812153927 dot GK3335 at redhat dot com> <55CB8230 dot 7080509 at moene dot org> <55CB8361 dot 7030507 at redhat dot com> <20150812174034 dot GL3335 at redhat dot com>

On 08/12/2015 11:40 AM, Marek Polacek wrote:

On Wed, Aug 12, 2015 at 11:33:21AM -0600, Jeff Law wrote:

On 08/12/2015 11:28 AM, Toon Moene wrote:

https://gcc.gnu.org/ml/gcc-testresults/2015-08/msg01036.html

[ Yes, that's at run time, not compile time ... ]

Hoping some of those are fixed by the sext_hwi changes from Mikael.


I hope too ;).  But they have nothing to do with the -Wshift-overflow IMHO.

Right. Toon was just pointing out that we're getting the left-shifterrors from ubsan all over the place. Those are totally independent ifyour -Wshift-overflow stuff.


Jeff

References:
- [C/C++ PATCH] Fix -Wshift-overflow with sign bit
  - From: Marek Polacek
- Re: [C/C++ PATCH] Fix -Wshift-overflow with sign bit
  - From: Toon Moene
- Re: [C/C++ PATCH] Fix -Wshift-overflow with sign bit
  - From: Jeff Law
- Re: [C/C++ PATCH] Fix -Wshift-overflow with sign bit
  - From: Marek Polacek

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