================
@@ -46,7 +50,7 @@ class SymbolRegionValue : public SymbolData {
 
   friend class SymExprAllocator;
   SymbolRegionValue(SymbolID sym, const TypedValueRegion *r)
-      : SymbolData(SymbolRegionValueKind, sym), R(r) {
+      : SymbolData(ClassKind, sym), R(r) {
----------------
NagyDonat wrote:

What is the reason for refactoring the internal machinery of the `classof()` 
methods? Does the rest of this commit depend on these changes?

As both these changes and the rest of the commit are large, I feel that perhaps 
it would be better to put these into a separate NFC change, but I can 
understand and accept if you don't want to bother with the rebasing. 

https://github.com/llvm/llvm-project/pull/144327
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to