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

Reply via email to