[cfe-commits] [Microsoft C++ ABI] Add initial support for array cookies

John McCall rjmccall at apple.com
Mon Apr 30 22:31:14 PDT 2012


On Apr 23, 2012, at 7:14 AM, Timur Iskhodzhanov wrote:
> It allows compiling "delete [] array;" code.
> It works exactly as the VS-generated cookies, except for the most
> simple case when the array is used for a type without a destructor
> (see the related FIXME in the test).

r155886.

I changed this to use size_t instead of assuming 32 bits.  I assume
Win64 uses a 64-bit cookie.  It'd be nice if you could verify this.

I also did a lot of other refactoring that shouldn't really be read as
comments on your code.

My implementation doesn't seem to have the problem you saw
when allocating arrays without destructors, so I went ahead and
added a test case for that.

Your test case named some local values (%3) and some types;  try
not to do that.

John.



More information about the cfe-commits mailing list