================
@@ -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

Reply via email to