martong added a comment. In D111640#3058304 <https://reviews.llvm.org/D111640#3058304>, @steakhal wrote:
> It looks good. > Do you think we could remove the `RangedConstraintManager` abstraction? Definitely. What's more, we could split out cohesive implementation details like `RangeSet`, `EquivalenceClass` from `RangeConstraintManager.cpp` into their own implementation files. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111640/new/ https://reviews.llvm.org/D111640 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits