<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Jan 28, 2014 at 10:16 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br>




<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">




<div>On Tue, Jan 28, 2014 at 10:01 PM, Richard Trieu <span dir="ltr"><<a href="mailto:rtrieu@google.com" target="_blank">rtrieu@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">GCC's -Waddress doesn't catch this or (&x == 0).  GCC's -Waddress does catch (x) for arrays, and (&x) for all types, but Clang doesn't.  </div>





</blockquote><div><br></div></div><div>Do you think your work would generalize/simplify to these cases? (even if it does, depending on how much work that is, it might still be worth doing this limited form you've done already - if it'd be easier to review it in two steps (if the second step is a logical extension to the first) rather than just reviewing the more general form)</div>




<div><div>
<div> </div></div></div></div></div></div></blockquote>Not sure at the moment.  I'll have a look at uniting and simplifying the cases.</div></div></div>