[cfe-commits] r65751 - in /cfe/trunk: lib/CodeGen/CGBlocks.cpp test/CodeGenObjC/blocks-unsupported.m

[Chris Lattner]

On Mar 1, 2009, at 4:32 PM, Mike Stump wrote:

> On Mar 1, 2009, at 3:39 PM, Anders Carlsson wrote:
>>> Yes, known issue.  We don't support Block_copy in it's full
>>> generality yet.  I'll probably be doing that next week.  The
>>> aforementioned projects don't use Block_copy in this way.
>>
>> If __block variables don't work with Block_copy, then I can't see
>> how they are not an unsupported feature (Especially not since you're
>> going to get miscompiles and potentially crashes in generated code).
>
> If the call to Block_copy said unsupported, the code given, would not
> compile.  It is Block_copy that is unsupported just yet.

Block_copy is not a compiler feature, __block is.  If the library  
version of Block_copy doesn't work right for __block variables, that  
is a failure of the compiler.

-Chris