george.karpenkov added a comment. > I could also move RangedConstraintManager.h under include
We probably don't want to do that: currently it can only be imported by files in `Core`, and we probably should keep it that way ================ Comment at: lib/StaticAnalyzer/Core/RangedConstraintManager.h:130 +template <> +struct ProgramStateTrait<ConstraintRange> + : public ProgramStatePartialTrait<ConstraintRangeTy> { ---------------- Why not also `REGISTER_TRAIT_WITH_PROGRAMSTATE` here? Repository: rC Clang https://reviews.llvm.org/D45920 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits