<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jun 28, 2016 at 4:44 PM, David Li <span dir="ltr"><<a href="mailto:davidxl@google.com" target="_blank" class="cremed">davidxl@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">davidxl added a comment.<br>
<br>
thanks. I don't seem to find explicit test cases added for this change. can you add one ?<br></blockquote><div><br></div><div>There is such a test already. test/CodeGen/PowerPC/tail-dup-layout.ll</div><div>The reason it wasn't added is that previously that test required the -outline-optional-branches flag, and now it doesn't. It's the prime example of what I'm trying to accomplish.</div><div>I'll add one to test the loop portion.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Please also update the description with a real motivation example -- the original code and the pseudo code after the transformation.<br></blockquote><div><br></div><div>Will do. It will likely be the example from the test mentioned above.</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/D20505" rel="noreferrer" target="_blank" class="cremed">http://reviews.llvm.org/D20505</a><br>
<br>
<br>
<br>
</blockquote></div><br></div></div>