[PATCH] D80905: [analyzer] Introduce weak dependencies to express *preferred* checker callback evaluation order

Kristóf Umann via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 8 04:52:22 PDT 2020


Szelethus marked 2 inline comments as done.
Szelethus added a comment.

Gentle ping



================
Comment at: clang/include/clang/StaticAnalyzer/Checkers/CheckerBase.td:141-142
+/// Describes preferred registration and evaluation order in between checkers.
+/// Unlike strong dependencies, this expresses dependencies in between
+/// diagnostics, and *not* modeling. In the case of an unsatisfied (disabled)
+/// weak dependency, the dependent checker might still be registered. If the
----------------
NoQ wrote:
> I wouldn't mind having predictable callback evaluation order for modeling as well. What's causing you to drop this scenario?
D80905#2066219


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80905





More information about the cfe-commits mailing list