Charusso accepted this revision. Charusso added a comment. Document!!4!44! It is great you have started to limit the notes, thanks!
================ Comment at: clang/include/clang/StaticAnalyzer/Core/BugReporter/BugReporterVisitors.h:110 + ConditionTracking +}; + ---------------- What about the following? ``` enum class TrackKind { Full, // comment what it does Condition // comment what it does }; ``` Please consider the following example by unspoken naming conventions: ``` enum class ColoringKind { RedColoring, BlueColoring } ``` would be ``` enum class Color { Red, Blue } ``` where each trivial what is does, by name. Please also note that, the thoroughness not working with redecls, assumptions, loop conditions, anything we failed to inline, etc... so it is not really that full tracking. But in our capabilities, it is the full what we could do. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64270/new/ https://reviews.llvm.org/D64270 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits