<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Oct 14, 2013 at 4:59 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">> I haven't looked in detail at your change, but PR14610 (perhaps poorly<br>
> explained by me in the bug) is more than about the duplicate debug info, but<br>
> about the fact that we implement C1/C2 and D1/D2 by having one call the<br>
> other so when you break on the ctor or dtor you actually break twice (once<br>
> inside *1, then in *2 or whichever way it goes).<br>
><br>
> By using aliases we get closer to (or the same as, in this instance) GCC's<br>
> behavior where there's only one call on the stack rather than one ctor<br>
> calling another ctor.<br>
><br>
> I think it'd be fine to just remove the XFAIL (& update the bug to mention<br>
> this). Might be worth mentioning that, if there are conditions where C1/C2<br>
> can't be aliases, this bug will likely manifest there.<br>
<br>
</div>I agree that this is what is happening. My change doesn't fix that PR,<br>
it just avoids the bug in the case the destructors can be aliased.<br>
<br>
Any thoughts on the patch itself?</blockquote><div><br></div><div>Little outside my domain/time right now - I've no opinion on the patch either way. For the debug side of things, feel free to/please remove the XFAIL if/when you commit this.<br>
<br>- David </div></div></div></div>