[PATCH] D105692: [analyzer][solver][NFC] Introduce ConstraintAssignor

Valeriy Savchenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 9 09:00:49 PDT 2021


vsavchenko added inline comments.


================
Comment at: clang/lib/StaticAnalyzer/Core/RangeConstraintManager.cpp:1388
+//===----------------------------------------------------------------------===//
+//                            New constraint handler
+//===----------------------------------------------------------------------===//
----------------
martong wrote:
> `New` now, but it might be old after a while.
Whoops, this is how I first named the class.  It is not new handler, but new constraint.  Because of this ambiguity I actually renamed it 😅


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105692



More information about the cfe-commits mailing list