[cfe-dev] Proposal: Type-Aware Memory Profiling

Dai Mikurube dmikurube at chromium.org
Thu Jan 17 00:16:20 PST 2013


Hello, Dmitri and cfe-dev.

I've finally added a document and tests in the patch
http://llvm-reviews.chandlerc.com/D298.  I think it's almost ready for
formal code review!

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
  * AST/DeclCXX, ExprCXX
  * CodeGen/CGCXXABI, CGClass, CGExprCXX
  * Sema/SemaDeclCXX, SemaExprCXX


Thanks in advance.


On Wed, Jan 16, 2013 at 2:42 PM, Dai Mikurube <dmikurube at chromium.org>wrote:

> Thanks for your comments, Dmitri.
>
> I just updated the patch and added non-working test files in the patch.
>  I'll add a document and tests soon.
> (It may take some time since I'm a stranger in Clang/LLVM testing.)
>
>
> On Wed, Jan 16, 2013 at 2:59 AM, Dmitri Gribenko <gribozavr at gmail.com>wrote:
>
>> On Tue, Jan 15, 2013 at 11:35 AM, Dai Mikurube <dmikurube at chromium.org>
>> wrote:
>> > Could anyone tell me a good document about Clang's test?  My patch
>> actually
>> > doesn't have tests yet.  I'm not sure how to add new tests in Clang's
>> > style...  Finally, I'd like to make sure that
>> > https://gist.github.com/4535834 works as its comment.
>>
>> For this patch CodeGen tests are needed.
>> test/CodeGen/builtins-multiprecision.c is a good example.
>>
>> Driver tests are also needed to ensure that the driver passes the flag
>> to the frontend.  test/Driver/retain-comments-from-system-headers.c is
>> about as much as needed -- just copy it and change the flag.
>>
>> Sema tests might also be needed, but I can not think of any.
>>
>> Dmitri
>>
>> --
>> main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
>> (j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/
>>
>
>
>
> --
> Dai MIKURUBE
>    dmikurube at chromium.org
>



-- 
Dai MIKURUBE
   dmikurube at chromium.org
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20130117/19e0b95d/attachment.html>


More information about the cfe-dev mailing list