[cfe-commits] r136161 - in /cfe/trunk: include/clang/Basic/DiagnosticSemaKinds.td lib/Sema/SemaExprCXX.cpp test/SemaCXX/new-delete.cpp

[Eli Friedman]

On Tue, Jul 26, 2011 at 3:28 PM, Sean Hunt <scshunt at csclub.uwaterloo.ca> wrote:
> On Tue, Jul 26, 2011 at 15:25, Eli Friedman <eli.friedman at gmail.com> wrote:
>> Author: efriedma
>> Date: Tue Jul 26 17:25:31 2011
>> New Revision: 136161
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=136161&view=rev
>> Log:
>> A couple minor issues with Sema for delete:
>>
>> 1. Attempting to delete an expression of incomplete class type should be an error, not a warning.
>>
>> 2. If someone tries to delete a pointer to an incomplete class type, make sure we actually emit
>> the delete expression after we warn.
>
> Deleting an object of incomplete class type is well-formed;
> [expr.delete]/5. This should not be an error.

 [expr.delete]/1: "The operand shall have a pointer to object type, or
a class type having a single non-explicit conversion function (12.3.2)
to a pointer to object type."  An expression of incomplete class type
isn't either of those.

-Eli