- User Since
- Oct 1 2018, 7:44 AM (46 w, 2 d)
I think this looks good. Maybe the tests should be extended to test auto as function return type, and if there's some special handling around decltype(auto), then it should be tested too, but I'm not sure it's actually needed here. What do you think?
Mon, Aug 19
Thanks for addressing my comments.
Thu, Aug 15
The overall structure seems alright. I'll let people more familiar with the code-base judge whether other important features should be added there as well.
Tue, Jul 30
LGTM, thanks for the update.
In vector_literals_nested.cl, we have tests for (global) constants. Do you think it would be worth testing those as well in C++ mode? Maybe the two files (vector_literals_nested.cl and vector_literals_valid.cl) should also be merged as most of their content seems duplicated.
Jul 22 2019
FYI In order to fix buildbot test failures, I've pushed https://reviews.llvm.org/rG1b2da771f561affe36eb5eb0c7a3d2862c5a5c1c. I'll keep an eye on buildbots for additional fallout.
Jul 19 2019
- Add minimal regression test for PR42665
- Add CXXMemberCallExpr::getObjectType()
Jul 18 2019
Add patch for Bug PR42665.
While investigating PR42665, I've noticed that getImplicitObjectArgument doesn't always return a pointer type. For example, when dealing with shared_ptr. In libc++'s test std/utilities/memory/util.smartptr/util.smartptr.shared/util.smartptr.shared.create/make_shared.pass.cpp, dumping CE in EmitVirtualDestructorCall shows that the callee is .~Bar (mind the .):
CXXMemberCallExpr 0x29d10a0 'void' `-MemberExpr 0x29d1070 '<bound member function type>' .~Bar 0x23f5250 `-CXXMemberCallExpr 0x29d1040 'struct Bar':'struct Bar' lvalue `-MemberExpr 0x29d1010 '<bound member function type>' .second 0x2883328 `-MemberExpr 0x29d0f80 '__compressed_pair<class std::__1::allocator<struct Bar>, struct Bar>':'class std::__1::__compressed_pair<class std::__1::allocator<struct Bar>, struct Bar>' lvalue ->__data_ 0x2896960 `-CXXThisExpr 0x29d0f70 'class std::__1::__shared_ptr_emplace<struct Bar, class std::__1::allocator<struct Bar> > *' implicit this
- Minor refactoring of getThisObjectType
- Removed unnecessary assertion
Jul 17 2019
- Addressed comments
Beside my two comments, I think this looks good.
Jul 16 2019
This should address the minor refactoring requesting in D64083.
Mind the fact that I've rebased the changes onto a more recent master version. If you look at the diff of v1 against v2 you might see some unrelated changes.
- Refactored common bits from CheckConstructorDeclarator and CheckDestructorDeclarator.
- Added as many "ThisTy" parameter I could.
- Addressed issue with identifier format in tests (%N to %var.ascast).
Jul 12 2019
Jul 11 2019
Jul 9 2019
Here are a few comments from me but keep in mind that English is not my primary language
Jul 4 2019
I'll commit this along with D64083 in a few days in case John has any feedback.
Addressed missing colon in FIXME comment.
Jul 2 2019
I added the FIXME and reduced the amount of testing. Let me know if it looks alright.
Address comments: reduce testing & add FIXME.
Jun 19 2019
When compiling this I get the following warning:
clang/lib/StaticAnalyzer/Core/ExprEngine.cpp:1213:10: warning: enumeration value 'CXXAddrspaceCastExprClass' not handled in switch
Jun 4 2019
May 29 2019
Dec 3 2018
Nov 30 2018
Please see https://reviews.llvm.org/D55136 for the patch addressing these issues.
Thank you for the detailed review. I'll work on a patch and add you as reviewer once done (prob. on Monday though).
Nov 27 2018
Nov 13 2018
Nov 1 2018
LGTM, but please add a comment in the test file.
Oct 22 2018
Oct 19 2018
Oct 4 2018
Note: the get_kernel_* functions used here all return unsigned integers, hence the warning/present patch.