================ @@ -90,8 +90,8 @@ namespace { if (A.isEmpty()) return B; - for (typename SET::iterator it = B.begin(), ei = B.end(); it != ei; ++it) { - A = A.add(*it); + for (const auto &element : B) { ---------------- Xazax-hun wrote:
We have pointers in these sets. Maybe iterating by value would be better to avoid double indirection? https://github.com/llvm/llvm-project/pull/157670 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits