[PATCH] D39543: [analyzer] Document the issue hash debugging facility

Gábor Horváth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 2 06:06:09 PDT 2017


xazax.hun added inline comments.


================
Comment at: docs/analyzer/DebugChecks.rst:255
+      int x = 1;
+      clang_analyzer_hashDump(x); // Hashed string of x on stderr.
+    }
----------------
NoQ wrote:
> Unlike `printState` and like all other functions, your function doesn't dump to plain stderr, but generates warnings instead. I didn't want `printState` to produce huge warnings because matching them with `expected-warning` would be more disgusting than `FileCheck`ing, but that's not your case. So i think you'd like to document it similarly to other functions, probably as `// expected-warning{{'sample output'}}`.
Yeah, sorry for the oversight. :) I wanted to get the documentation part done as soon as possible to work on other parts. But it looks like these kinds of tasks should have the same level of attention as others. 


https://reviews.llvm.org/D39543





More information about the cfe-commits mailing list