[PATCH] D53076: [analyzer] ConditionBRVisitor: enhance to write out more information

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 28 17:32:09 PDT 2019


NoQ added a comment.

Instead of having those as events similar to "Assuming", we could turn the new "Knowing" pieces into floating pop-ups - imagine you hover the mouse over a condition `foo()` and it says "`foo()` is false". That is, instead of `PathDiagnosticsEventPiece`, a new kind of piece can be introduced that is shown in scan-build as a pop-up similar to macro pop-ups. Like this:

F8556875: mock.png <https://reviews.llvm.org/F8556875>

We already have infrastructure for hover pop-up hints in HTML reports - we use it to display macro expansions, so it could be something similar, just of a different color.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D53076/new/

https://reviews.llvm.org/D53076





More information about the cfe-commits mailing list