diff options
| author | Volodymyr Sapsai <vsapsai@apple.com> | 2018-12-20 22:43:26 +0000 |
|---|---|---|
| committer | Volodymyr Sapsai <vsapsai@apple.com> | 2018-12-20 22:43:26 +0000 |
| commit | 232d22f380eb1e9a9a1e71500b90d6f269a071b3 (patch) | |
| tree | fdcc77f0a0103a84f3b0d94c42d0d7fcd3f68ff3 /lldb/packages/Python/lldbsuite/test/sample_test/TestSampleInlineTest.py | |
| parent | ddfaf07526b79a097da51d805b58ea2214880113 (diff) | |
| download | bcm5719-llvm-232d22f380eb1e9a9a1e71500b90d6f269a071b3.tar.gz bcm5719-llvm-232d22f380eb1e9a9a1e71500b90d6f269a071b3.zip | |
[CodeGen] Fix assertion on emitting cleanup for object with inlined inherited constructor and non-trivial destructor.
Fixes assertion
> Assertion failed: (isa<X>(Val) && "cast<Ty>() argument of incompatible type!"), function cast, file llvm/Support/Casting.h, line 255.
It was triggered by trying to cast `FunctionDecl` to `CXXMethodDecl` as
`CGF.CurCodeDecl` in `CallBaseDtor::Emit`. It was happening because
cleanups were emitted in `ScalarExprEmitter::VisitExprWithCleanups`
after destroying `InlinedInheritingConstructorScope`, so
`CodeGenFunction.CurCodeDecl` didn't correspond to expected cleanup decl.
Fix the assertion by emitting cleanups before leaving
`InlinedInheritingConstructorScope` and changing `CurCodeDecl`.
Test cases based on a patch by Shoaib Meenai.
Fixes PR36748.
rdar://problem/45805151
Reviewers: rsmith, rjmccall
Reviewed By: rjmccall
Subscribers: jkorous, dexonsmith, cfe-commits, smeenai, compnerd
Differential Revision: https://reviews.llvm.org/D55543
llvm-svn: 349848
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/sample_test/TestSampleInlineTest.py')
0 files changed, 0 insertions, 0 deletions
