diff options
| author | Mike Stump <mrs@apple.com> | 2009-03-25 17:58:24 +0000 |
|---|---|---|
| committer | Mike Stump <mrs@apple.com> | 2009-03-25 17:58:24 +0000 |
| commit | a803b0e6ca0d1c71a0b3f5d7380da8bbacf2f983 (patch) | |
| tree | a44333874d55cc0281e8c1b105408ff59ed49b72 /lib/CodeGen/CGBlocks.cpp | |
| parent | 65633f96c97af2553fad548180f199615d570c03 (diff) | |
Fixup codegen for block literals that bleed copy/dispose information
from previous block literals.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@67696 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen/CGBlocks.cpp')
| -rw-r--r-- | lib/CodeGen/CGBlocks.cpp | 15 |
1 files changed, 10 insertions, 5 deletions
diff --git a/lib/CodeGen/CGBlocks.cpp b/lib/CodeGen/CGBlocks.cpp index 5780fc2958..eba741bc3c 100644 --- a/lib/CodeGen/CGBlocks.cpp +++ b/lib/CodeGen/CGBlocks.cpp @@ -35,7 +35,8 @@ Enable__block("f__block", llvm::cl::init(true)); llvm::Constant *CodeGenFunction:: -BuildDescriptorBlockDecl(uint64_t Size, const llvm::StructType* Ty, +BuildDescriptorBlockDecl(bool BlockHasCopyDispose, uint64_t Size, + const llvm::StructType* Ty, std::vector<HelperInfo> *NoteForHelper) { const llvm::Type *UnsignedLongTy = CGM.getTypes().ConvertType(getContext().UnsignedLongTy); @@ -155,18 +156,20 @@ llvm::Value *CodeGenFunction::BuildBlockLiteralTmp(const BlockExpr *BE) { // __invoke uint64_t subBlockSize, subBlockAlign; llvm::SmallVector<const Expr *, 8> subBlockDeclRefDecls; + bool subBlockHasCopyDispose = false; llvm::Function *Fn = CodeGenFunction(CGM).GenerateBlockFunction(BE, Info, CurFuncDecl, LocalDeclMap, subBlockSize, subBlockAlign, subBlockDeclRefDecls, - BlockHasCopyDispose); + subBlockHasCopyDispose); + BlockHasCopyDispose |= subBlockHasCopyDispose; Elts[3] = Fn; if (!Enable__block && BlockHasCopyDispose) ErrorUnsupported(BE, "block literal that requires copy/dispose"); - if (BlockHasCopyDispose) + if (subBlockHasCopyDispose) flags |= BLOCK_HAS_COPY_DISPOSE; // __isa @@ -186,7 +189,8 @@ llvm::Value *CodeGenFunction::BuildBlockLiteralTmp(const BlockExpr *BE) { if (subBlockDeclRefDecls.size() == 0) { // __descriptor - Elts[4] = BuildDescriptorBlockDecl(subBlockSize, 0, 0); + assert(subBlockHasCopyDispose == false); + Elts[4] = BuildDescriptorBlockDecl(subBlockHasCopyDispose, subBlockSize, 0, 0); // Optimize to being a global block. Elts[0] = CGM.getNSConcreteGlobalBlock(); @@ -318,7 +322,8 @@ llvm::Value *CodeGenFunction::BuildBlockLiteralTmp(const BlockExpr *BE) { NoteForHelper.resize(helpersize); // __descriptor - llvm::Value *Descriptor = BuildDescriptorBlockDecl(subBlockSize, Ty, + llvm::Value *Descriptor = BuildDescriptorBlockDecl(subBlockHasCopyDispose, + subBlockSize, Ty, &NoteForHelper); Descriptor = Builder.CreateBitCast(Descriptor, PtrToInt8Ty); Builder.CreateStore(Descriptor, Builder.CreateStructGEP(V, 4, "block.tmp")); |