[PATCH] D50211: [analyzer] Fix displayed checker name for InnerPointerChecker

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 2 19:49:52 PDT 2018


NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.

I see, so that's how it's done!

I also noticed that checker name was weird in exploded graph dumps, i.e. it was showing regular new/delete stuff as if it was done by InnerPointer checker. I'll check if this is fixed tomorrow.



================
Comment at: lib/StaticAnalyzer/Checkers/MallocChecker.cpp:1685
+
+    return Optional<MallocChecker::CheckKind>();
+  }
----------------
It's usually just `None`, but i guess whoever touched this code before didn't know; it's pretty hard to discover.


Repository:
  rC Clang

https://reviews.llvm.org/D50211





More information about the cfe-commits mailing list