<div>My understanding of the current implementation and (limited) knowledge of LLVM IR  leads me to believe that if the current mem2reg was modifed to not be restricted to allocas in entry block then  it would still work correctly, and possibly catch more cases. I think that this is because the function IsAllocaPromotable(...) checks for direct loads and stores from the pointer returned by alloca and if the pointer is used in other ways it doesnt promote the loads and stores of the alloca pointer. So after scalar promotion of such loads and stores the alloca would be dead.<br>
 <br>Can someone confirm that or point out some case wher that wouldnt be correct.<br> <br>Thanks</div>
<div>Vinod<br></div>
<div class="gmail_quote">On Tue, Feb 26, 2013 at 8:39 AM, John Criswell <span dir="ltr"><<a href="mailto:criswell@illinois.edu" target="_blank">criswell@illinois.edu</a>></span> wrote:<br>
<blockquote style="BORDER-LEFT:#ccc 1px solid;MARGIN:0px 0px 0px 0.8ex;PADDING-LEFT:1ex" class="gmail_quote">
<div class="im">On 2/26/13 8:06 AM, Duncan Sands wrote:<br>
<blockquote style="BORDER-LEFT:#ccc 1px solid;MARGIN:0px 0px 0px 0.8ex;PADDING-LEFT:1ex" class="gmail_quote">Hi Justin,<br><br>
<blockquote style="BORDER-LEFT:#ccc 1px solid;MARGIN:0px 0px 0px 0.8ex;PADDING-LEFT:1ex" class="gmail_quote">    an alloca outside of the entry block might be inside a loop, in which case the<br>    semantics are that it would allocate more stack space on every loop iteration.<br>
    I think some of the optimizers that run later try to move allocas into the entry<br>    block if possible, but in general it is simpler to have the front-end just put<br>    them there in the first place.<br><br><br>Mem2reg is already changing that semantic, though.  If I use an "alloca i32" in<br>
the entry block, then I am saying I want 4 bytes of stack space, but mem2reg may<br>replace that with registers.<br></blockquote><br>the problem isn't with mem2reg changing the amount of used stack space, it's<br>
that those semantics get in the way of the mem2reg transform.  For example,<br>as you get new stack space each time round the loop, a write to the alloca<br>can't be retrieved by reading it back from the alloca next time round the<br>
loop, because it isn't the same memory.  This is quite different to how things<br>work if the alloca is in the entry block.<br></blockquote><br></div>I think it might be useful to separate what is possible versus what is done for practicality.<br>
<br>I believe that there is no technical reason for mem2reg to restrict itself to allocas within the entry block.  It should be possible to promote allocas not within the entry block into SSA registers provided that they meet certain restrictions.  Off the top of my head, those restrictions are ensuring that the alloca is not in a loop and ensuring that the returned pointer doesn't escape the function.  I suspect that allocas that don't dominate all of the basic blocks in the function might require special handling.<br>
<br>My guess is that mem2reg limits itself to allocas within the entry block because that is where nearly all allocas that can be converted reside, and there's little benefit (at least for C/C++) in trying to promote allocas that aren't in the entry block.  Mem2reg was originally built so that front-ends (namely the original llvm-gcc) wouldn't have to do SSA construction; since llvm-gcc put its allocas in the entry block, there probably wasn't a need to promote other allocas.<br>
<br>If you want to promote allocas outside the entry block, you can probably implement an algorithm to do it.  I think mem2reg doesn't do it because it hasn't been worth the trouble.<br><br>-- John T. 
<div class="HOEnZb">
<div class="h5"><br><br>
<blockquote style="BORDER-LEFT:#ccc 1px solid;MARGIN:0px 0px 0px 0.8ex;PADDING-LEFT:1ex" class="gmail_quote"><br>Ciao, Duncan.<br>______________________________<u></u>_________________<br>LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu" target="_blank">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu/" target="_blank">http://llvm.cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvmdev</a><br>
</blockquote><br>______________________________<u></u>_________________<br>LLVM Developers mailing list<br><a href="mailto:LLVMdev@cs.uiuc.edu" target="_blank">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu/" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvmdev</a><br></div></div></blockquote></div><br>