================
@@ -359,7 +326,80 @@ class RegionBindingsRef : public 
llvm::ImmutableMapRef<const MemRegion *,
 };
 } // end anonymous namespace
 
+/// This class represents the same as \c RegionBindingsRef, but with a limit on
+/// the number of bindings that can be added.
+class BoundedRegionBindingsRef : public RegionBindingsRef {
----------------
NagyDonat wrote:

`<bikeshedding>`
Perhaps use `Limited` instead of `Bounded` in the name of this class, because 
the common linguistical origin of the words "Bounded" and "Binding" makes the 
current name a bit hard to parse. (I almost read "Bound" region bindings and 
felt that it's tautological that "bindings" are "bound".)

However, this is a very "soft" request, feel free to keep the current name if 
you prefer it.
`</bikeshedding>`

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

Reply via email to