<div dir="ltr">Hello, Dmitri and cfe-dev.<div><br></div><div style>I've finally added a document and tests in the patch <a href="http://llvm-reviews.chandlerc.com/D298">http://llvm-reviews.chandlerc.com/D298</a>.  I think it's almost ready for formal code review!</div>

<div style><br></div><div style>I wonder if someone could recommend reviewers good for the patch.  I'm not sure who are good to ask...  It changes around C++ memory allocation especially in</div><div style>  * AST/DeclCXX, ExprCXX</div>

<div style>  * CodeGen/CGCXXABI, CGClass, CGExprCXX</div><div style>  * Sema/SemaDeclCXX, SemaExprCXX</div><div style><br></div><div style><br></div><div style>Thanks in advance.</div></div><div class="gmail_extra"><br><br>

<div class="gmail_quote">On Wed, Jan 16, 2013 at 2:42 PM, Dai Mikurube <span dir="ltr"><<a href="mailto:dmikurube@chromium.org" target="_blank">dmikurube@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div dir="ltr"><div class="gmail_default">Thanks for your comments, Dmitri.</div><div class="gmail_default"><br></div><div class="gmail_default">I just updated the patch and added non-working test files in the patch.  I'll add a document and tests soon.</div>


<div class="gmail_default">(It may take some time since I'm a stranger in Clang/LLVM testing.)</div></div><div class="gmail_extra"><div><div class="h5"><br><br><div class="gmail_quote">On Wed, Jan 16, 2013 at 2:59 AM, Dmitri Gribenko <span dir="ltr"><<a href="mailto:gribozavr@gmail.com" target="_blank">gribozavr@gmail.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>On Tue, Jan 15, 2013 at 11:35 AM, Dai Mikurube <<a href="mailto:dmikurube@chromium.org" target="_blank">dmikurube@chromium.org</a>> wrote:<br>



> Could anyone tell me a good document about Clang's test?  My patch actually<br>
> doesn't have tests yet.  I'm not sure how to add new tests in Clang's<br>
> style...  Finally, I'd like to make sure that<br>
> <a href="https://gist.github.com/4535834" target="_blank">https://gist.github.com/4535834</a> works as its comment.<br>
<br>
</div>For this patch CodeGen tests are needed.<br>
test/CodeGen/builtins-multiprecision.c is a good example.<br>
<br>
Driver tests are also needed to ensure that the driver passes the flag<br>
to the frontend.  test/Driver/retain-comments-from-system-headers.c is<br>
about as much as needed -- just copy it and change the flag.<br>
<br>
Sema tests might also be needed, but I can not think of any.<br>
<span><font color="#888888"><br>
Dmitri<br>
<br>
--<br>
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if<br>
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <<a href="mailto:gribozavr@gmail.com" target="_blank">gribozavr@gmail.com</a>>*/<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div></div></div><span class="HOEnZb"><font color="#888888">-- <br><div>Dai MIKURUBE</div><div>   <a href="mailto:dmikurube@chromium.org" target="_blank">dmikurube@chromium.org</a></div>


</font></span></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Dai MIKURUBE</div><div>   <a href="mailto:dmikurube@chromium.org" target="_blank">dmikurube@chromium.org</a></div>
</div>