This revision was automatically updated to reflect the committed changes.
Closed by commit rL315300: [analyzer] MisusedMovedObject: Add printState()
method for self-debugging. (authored by dergachev).
Changed prior to commit:
https://reviews.llvm.org/D31541?vs=93661&id=118350#toc
Repository:
xazax.hun accepted this revision.
xazax.hun added a comment.
LGTM!
https://reviews.llvm.org/D31541
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
NoQ added a comment.
Hmm, i've been thinking of writing a test for this via `ExprInspection`'s
`clang_analyzer_printState()`, however `printState()` functionality is only
enabled in debug builds, and i'm not seeing how to enable the test only on
debug builds (there's `REQUIRES:` but it only see
szepet accepted this revision.
szepet added a comment.
This revision is now accepted and ready to land.
LGTM, thank you (again)!
https://reviews.llvm.org/D31541
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/ma
NoQ created this revision.
The `printState()` method adds the checker's information to the
`State->dump()`. I find it very handy when debugging by looking at the exploded
graphs, so i think it's great to have this method implemented in all checkers.
It helped me already.
I think it's not a pro
