alexfh added a comment.

Does this just fix the tests broken with http://reviews.llvm.org/D20498 or have 
you looked at other checks that might be broken by 
http://reviews.llvm.org/D20498, but don't have relevant tests? In the latter 
case we'd certainly need your help figuring out where further problems may 
appear, e.g. what specific patterns should we look for in the AST matchers used 
in clang-tidy checks or other Clang tools. Does this boil down to `expr(X)` -> 
`expr(ignoringExprWithCleanups(X))`?


http://reviews.llvm.org/D21243



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to