[PATCH] D75842: [Analyzer] Bugfix for CheckerRegistry

Balogh, Ádám via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 9 06:57:38 PDT 2020


baloghadamsoftware added a comment.

In D75842#1912293 <https://reviews.llvm.org/D75842#1912293>, @xazax.hun wrote:

> I am not sure if I would call this a bugfix. Enabling a checker without one of its dependency will potentially cause the checker to misbehave. I am uncomfortable changing the current behavior to a more dangerous one without any diagnostics. Including the diagnostic with this patch should not make this too big.


This //is// a bugfix. The current behavior is wrong due to a mistake. A checker is registered which should not and this may cause crashes.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75842





More information about the cfe-commits mailing list