<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">-cfe-commits, +cfe-dev (sorry, /me unable to handle mailing lists)</div><div class="gmail_quote"><br></div><div class="gmail_quote">On Tue, Jul 29, 2014 at 3:51 PM, Manuel Klimek <span dir="ltr"><<a href="mailto:klimek@google.com" target="_blank">klimek@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">Hi Jordan,<div><br></div><div>I am working on a fix for lifetime extended temporaries, and I have a hard time understanding some of the code.</div>
<div>1. addLocalScopeForVarDecl already has a FIXME; I have no idea how to apply that though - it seems like we always add the VarDecl to the scope, even if the type we look at is from the lifetime extended temporary; as noted, we need to be able to handle the scope of multiple lifetime extended temporaries; do we need a completely different approach here then?</div>

<div>2. addAutomaticObjDtors seems to work on what addLocalScopeForVarDecl does, so it seems like we'll need to adapt this after solving (1)</div><div><br></div><div>My main problem is that I don't have enough insight into what else the local scope is used for, so I'm not sure how we'd want to change it to account for lifetime extended temporaries.</div>
</div></blockquote><div><br></div><div>After some investigation, it looks to me like we'll want to add some way to CFGAutomaticObjDtor to allow not only specifying VarDecls, but somehow directly referencing the lifetime extended expressions (?)</div>
<div><br></div></div></div></div>