summaryrefslogtreecommitdiffstats
path: root/lldb/packages/Python/lldbsuite/test/python_api/default-constructor
diff options
context:
space:
mode:
authorRaphael Isemann <teemperor@gmail.com>2018-08-22 22:50:45 +0000
committerRaphael Isemann <teemperor@gmail.com>2018-08-22 22:50:45 +0000
commit3142a49068f7be1275e4202d745e48090b39745a (patch)
treea44d1748c1b7df6203214bfe5cf2e8b1e435504a /lldb/packages/Python/lldbsuite/test/python_api/default-constructor
parentff862be4ef99862f2dba065ebfefb46e4f80fae1 (diff)
downloadbcm5719-llvm-3142a49068f7be1275e4202d745e48090b39745a.tar.gz
bcm5719-llvm-3142a49068f7be1275e4202d745e48090b39745a.zip
[ASTImporter] Actually test ArrayInitLoopExpr in the array-init-loop-expr test.
Summary: The `array-init-loop-expr` test is currently not testing the importing of ArrayInitLoopExprs. This is because we import the `S` struct into the `test.cpp` context and only do a copy-assignment in `test.cpp`, so the actual ArrayInitLoopExpr we wanted to import is generated by clang directly in the target context. This means we actually never test the importing of ArrayInitLoopExpr with this test, which becomes obvious when looking at the missing test coverage for the respective VisitArrayInitLoopExpr method. This patch moves the copy-assignment of our struct to the `S.cpp` context, which means that `test.cpp` now actually has to import the ArrayInitLoopExpr. Reviewers: a.sidorin, a_sidorin Reviewed By: a_sidorin Subscribers: a_sidorin, martong, cfe-commits Differential Revision: https://reviews.llvm.org/D51115 llvm-svn: 340467
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/python_api/default-constructor')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud