<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Mon, Aug 1, 2016 at 5:51 PM Duncan P. N. Exon Smith via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I just want to note that there's currently a miscompile in C code with lifetime markers:<br>
  <a href="http://lists.llvm.org/pipermail/cfe-dev/2016-July/050066.html" rel="noreferrer" target="_blank">http://lists.llvm.org/pipermail/cfe-dev/2016-July/050066.html</a><br>
<br>
Depending on how we fix that, we could end up being more pessimistic about when markers are emitted.  I suggest we fix that before benchmarking this change.<br></blockquote><div><br></div><div>That shouldn't impact this change.</div><div><br></div><div>The inliner will emit *inlining* based lifetime markers all on its own for any static allocas that the frontend doesn't annotate. So we could turn off all lifetime markers in Clang and the merging here should still not be necessary to get the stack savings.</div><div><br></div><div>(And I have now verified that yes, the inliner inserts these markers...)</div></div></div>