[PATCH] D52133: [analyzer] A testing facility for testing relationships between symbols.

George Karpenkov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 17 13:17:19 PDT 2018


george.karpenkov accepted this revision.
george.karpenkov added inline comments.
This revision is now accepted and ready to land.


================
Comment at: lib/StaticAnalyzer/Checkers/ExprInspectionChecker.cpp:273
+
+  for (auto I : State->get<DenotedSymbols>()) {
+    SymbolRef Sym = I.first;
----------------
Sidenote: this code fragment is so common I wonder whether we should have a helper/facility for that.


https://reviews.llvm.org/D52133





More information about the cfe-commits mailing list