[clang] [analyzer] Remove overzealous "No dispatcher registered" assertion (PR #107294)

via cfe-commits cfe-commits at lists.llvm.org
Sun Sep 8 16:19:39 PDT 2024


================
@@ -48,15 +48,7 @@ bool CheckerManager::hasPathSensitiveCheckers() const {
       EvalCallCheckers, EndOfTranslationUnitCheckers);
 }
 
-void CheckerManager::finishedCheckerRegistration() {
-#ifndef NDEBUG
-  // Make sure that for every event that has listeners, there is at least
-  // one dispatcher registered for it.
-  for (const auto &Event : Events)
-    assert(Event.second.HasDispatcher &&
-           "No dispatcher registered for an event");
-#endif
-}
+void CheckerManager::finishedCheckerRegistration() {}
----------------
vabridgers wrote:

I resolved this comment with the latest update. 

https://github.com/llvm/llvm-project/pull/107294


More information about the cfe-commits mailing list