<div dir="ltr">On 26 August 2013 12:00, Puyan Lotfi <span dir="ltr"><<a href="mailto:plotfi@apple.com" target="_blank">plotfi@apple.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Test cases attached along with patch.<br></blockquote><div><br></div><div>The testcases should actually be part of the patch. If you're using svn, "svn add" the testcase files, and they'll show up when you run svn diff.</div>

<div><br></div><div>Nick</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Aug 26, 2013, at 8:06 AM, Meador Inge <<a href="mailto:meadori@codesourcery.com">meadori@codesourcery.com</a>> wrote:<br>


<br>
> On 08/26/2013 03:40 AM, Puyan Lotfi wrote:<br>
><br>
>> Could someone on the list review my changes, provide feedback, and if possible<br>
>> submit my changes?<br>
>><br>
>> I also have some test cases I've written but I am still trying to figure out how<br>
>> to add them to llvm/test/Transforms/GlobalOpt (I don't see a lit.local.cfg in<br>
>> that directory as the docs specify).<br>
><br>
> Folks can provide feedback given just the patch, but it will be easier to<br>
> review with the test cases as a part of the code change patch.  Also, it<br>
> shouldn't be committed without the test cases.  So, I would get the test case<br>
> changes worked out and re-submit.<br>
><br>
> --<br>
> Meador Inge<br>
> CodeSourcery / Mentor Embedded<br>
<br>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div></div>