[clang] [analyzer] Prevent crash due to missing EventDispatch in corner case (PR #107294)

[Donát Nagy via cfe-commits]

================
@@ -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() {}
----------------
NagyDonat wrote:

Let's just remove this method altogether -- there is no reason to keep it. (If somebody needs some "run this after checker registration" logic in the future, they can easily re-add something like this.)

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