[PATCH] D75514: [Analyzer] Only add container note tags to the operations of the affected container

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 5 07:40:47 PST 2020


NoQ added inline comments.


================
Comment at: clang/lib/StaticAnalyzer/Checkers/ContainerModeling.cpp:719-720
+      const auto *PSBR = dyn_cast<PathSensitiveBugReport>(&BR);
+      if (!PSBR)
+        return "";
+
----------------
Mmm, this is definitely impossible. We should change the `NoteTag` API to pass in a `PathSensitiveBugReport &`. My bad.


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

https://reviews.llvm.org/D75514





More information about the cfe-commits mailing list