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...)<div>
<div><br><div class="gmail_quote">On Thu, Jul 19, 2012 at 7:07 PM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">On Thu, Jul 19, 2012 at 7:06 PM, Nico Weber <<a href="mailto:thakis@chromium.org">thakis@chromium.org</a>> wrote:<br>
> Hi,<br>
><br>
> the attached patch lets GNUNullExpr::HasSideEffects() return false.<br>
> This fixes PR13413, -Wunused-private-field not warning on used private<br>
> fields that are initialized to NULL.<br>
><br>
> Nico<br>
</div></div><br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div></div>