george.karpenkov added a comment.
I think I lack context to completely get what is going on here: I assume we
don't model the assignment here?
================
Comment at: lib/StaticAnalyzer/Core/ExprEngineC.cpp:533
- if (isa<CXXConstructExpr>(Init)) {
+ if (isa<CXXConstructExpr>(Init) || isa<CXXStdInitializerListExpr>(Init)) {
// No work needed. Just pass the value up to this expression.
----------------
Empty "if" looks weird, why not just negate the condition, and then you could
get rid of "else"?
https://reviews.llvm.org/D39803
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
