[cfe-commits] [patch] PR13413: Let GNUNullExpr::HasSideEffects() return false
richard at metafoo.co.uk
Thu Jul 19 22:00:43 CDT 2012
LGTM. While you're there, can you also fix CXXNullPtrLiteralExpr,
CXXBoolLiteralExpr, and CXXThisExpr? I think those are the most important
ones we're getting wrong, for the purpose of this warning. (If you have the
energy, a whole big pile of other expressions are mishandled here,
including most of the C++-specific nodes. We should convert this code to a
switch so that people know to update it when adding a new Expr node...)
On Thu, Jul 19, 2012 at 7:07 PM, Nico Weber <thakis at chromium.org> wrote:
> On Thu, Jul 19, 2012 at 7:06 PM, Nico Weber <thakis at chromium.org> wrote:
> > Hi,
> > the attached patch lets GNUNullExpr::HasSideEffects() return false.
> > This fixes PR13413, -Wunused-private-field not warning on used private
> > fields that are initialized to NULL.
> > Nico
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits