<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On Apr 23, 2013, at 9:01 AM, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">This test looks really odd. It seems designed to be an execution test<br>(has a main function, uses assert), but we only produce IL from it. It<br>has been odd since it was added in r72437.<br><br>Douglas, what was the intent of this test? Should it be moved to the<br>llvm-testsuite where it will actually be executed?<br></div></blockquote><div><br></div><div>This test can simply be removed. It's not adding any value.</div><div><br></div></div><div><span class="Apple-tab-span" style="white-space:pre"> </span>- Doug</div><div><br><blockquote type="cite"><div style="letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">On 23 April 2013 11:44, Jyotsna Verma <<a href="mailto:jverma@codeaurora.org">jverma@codeaurora.org</a>> wrote:<br><blockquote type="cite">The attached patch modifies SemaTemplate/example-dynarray.cpp test to have<br>explicit declaration for malloc, free and assert functions and removes its<br>dependency on <stdlib.h> and <assert.h>.<br><br><br><br>Please let me know if the patch is okay to commit.<br><br><br><br>Thanks,<br><br>Jyotsna<br><br><br><br>--<br><br>Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by<br>The Linux Foundation<br><br><br><br><br>_______________________________________________<br>cfe-commits mailing list<br><a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</blockquote></div></blockquote></div><br></body></html>