<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Mar 26, 2014 at 3:40 PM, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class="">On Wed, Mar 26, 2014 at 3:10 PM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</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"><div class="gmail_quote">

<div>On Wed, Mar 26, 2014 at 9:36 AM, Stephan Tolksdorf <span dir="ltr"><<a href="mailto:st@quanttec.com" target="_blank">st@quanttec.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">Hi Reid,<br>
<br>
Did you overlook that I had assigned PR19253 to myself and posted a patch to this list (D3190)? :-)<br></blockquote></div></div></div></div></blockquote><div><br></div></div><div>Sorry, I was reading my inbox, not commit mail.  This looked super obvious, so I went ahead.<br>

</div><div class=""><div> </div><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">

<div class="gmail_quote"><div><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">
I think that fixing HasIrrelevantDestructor (as I did in my patch) would be a better solution for this issue.</blockquote><div><br></div></div><div>I agree that we should fix HasIrrelevantDestructor. But... we shouldn't warn on a trivial destructor no matter whether it's public or whether it calls non-public destructors. hasIrrelevantDestructor is supposed to just be an optimization, and shouldn't affect our semantics. => We want both fixes :)</div>


<div><br></div><div>Another test case, should not warn (under either of the two 'global destructor' warnings):</div><div><br></div><div>class A {</div><div>  friend struct B;</div><div>  ~A() = default;</div><div>


};</div><div>struct B {</div><div>  ~B() = default;</div><div>} b;</div></div></div></div></blockquote><div><br></div></div><div>Is B supposed to inherit from A here?</div></div></div></div>
</blockquote></div><br></div><div class="gmail_extra">It was supposed to, yes :)</div></div>