[PATCH] D58894: [analyzer] Handle modification of vars inside an expr with comma operator

Djordje Todorovic via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Mar 3 23:43:42 PST 2019


djtodoro created this revision.
djtodoro added reviewers: shuaiwang, lebedev.ri.
Herald added subscribers: Charusso, jdoerfert, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, szepet, baloghadamsoftware, xazax.hun.

We should track mutation of a variable within a comma operator expression.
Current code in `ExprMutationAnalyzer `does not handle it.

This will handle cases like:

`(a, b) ++`  < == `b `is modified
`(a, b) = c` < == `b `is modifed


https://reviews.llvm.org/D58894

Files:
  include/clang/AST/Expr.h
  lib/Analysis/ExprMutationAnalyzer.cpp
  unittests/Analysis/ExprMutationAnalyzerTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D58894.189123.patch
Type: text/x-patch
Size: 5058 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190304/99ad4876/attachment.bin>


More information about the cfe-commits mailing list