<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 16, 2015 at 3:28 PM, Keno Fischer via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">loladiro added a comment.<br>
<br>
I'm not sure the added complexity of walking the chain and checking everything would be worth it. It seems like that would just be manually inlining  getInlinedAtScope + doing the memoization. </blockquote><div><br></div><div>Yep, something like that, I think (I forget how exactly the scope V inlinedAt scope all plays together here)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">It seems like a reasonable assumption that the scope declared on one instruction would also be declared on instructions nearby, while that's not necessarily the case for intermediate ones (plus if it is we'll put it in the list when we get there).<br></blockquote><div><br></div><div>Not quite following - what I'm not sure is why it's nice to cache the immediate scope, but not intermediate ones - reasonably nested code may have many nested scopes (big switches from tablegen, for example?) with an immediately shared outer scope that's still nesnted within the function scope.<br><br>Just seems a bit quirky, that's all - I /think/ we should probably just skip caching the immediate scope (& assume the scope walk is cheap enough) or cache all of them.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<a href="http://reviews.llvm.org/D14697" rel="noreferrer" target="_blank">http://reviews.llvm.org/D14697</a><br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div></div>