diff options
| author | Vedant Kumar <vsk@apple.com> | 2015-12-21 19:30:37 +0000 |
|---|---|---|
| committer | Vedant Kumar <vsk@apple.com> | 2015-12-21 19:30:37 +0000 |
| commit | eb37ec87e48ea58c2a5555faeafc7968be5738bb (patch) | |
| tree | 6e3e2d3115cdd40ea4f79b53ea3f86980c082385 /clang/test/CodeGenObjCXX | |
| parent | b2910ea67e4d796cbed535edee114882691aba55 (diff) | |
| download | bcm5719-llvm-eb37ec87e48ea58c2a5555faeafc7968be5738bb.tar.gz bcm5719-llvm-eb37ec87e48ea58c2a5555faeafc7968be5738bb.zip | |
[CodeGen] Fix assignments of inline layouts into the byref structure
When using blocks, a byref structure is created to represent the
closure. The "byref.layout" field of this structure is an i8*. However,
some 'inline' layouts are represented as i64's, not i8*'s.
Prior to r246985 we cast the i64 'inline' layout to an i8* before
assigning it into the byref structure. This patch brings the cast back
and adds a regression test.
rdar://23713871
llvm-svn: 256185
Diffstat (limited to 'clang/test/CodeGenObjCXX')
| -rw-r--r-- | clang/test/CodeGenObjCXX/blocks.mm | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/clang/test/CodeGenObjCXX/blocks.mm b/clang/test/CodeGenObjCXX/blocks.mm index fd93437ff65..63a1b33f355 100644 --- a/clang/test/CodeGenObjCXX/blocks.mm +++ b/clang/test/CodeGenObjCXX/blocks.mm @@ -68,3 +68,18 @@ class CaptureThisAndAnotherPointer { takeBlock(^{ useValues(ptr, this); }); } }; + +// rdar://problem/23713871 +// Check that we don't crash when using BLOCK_LAYOUT_STRONG. +#pragma clang assume_nonnull begin +@interface NSUUID @end +#pragma clang assume_nonnull end + +struct Wrapper1 { NSUUID *Ref; }; +struct Wrapper2 { Wrapper1 W1; }; + +@implementation B +- (void) captureStrongRef { + __block Wrapper2 W2; +} +@end |

