[PATCH] D69417: [ADT] add equality operator for SmallSet

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 29 08:25:39 PDT 2019


fhahn added inline comments.


================
Comment at: llvm/include/llvm/ADT/SmallSet.h:264
+
+  for (auto &E : LHS)
+    if (!RHS.count(E))
----------------
nit: the whole remainder of the function could just be something like 
```
// All elements in LHS must also be in RHS
return all_of(LHS, [&RHS](T &E) {return RHS.count(E); });
```


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69417/new/

https://reviews.llvm.org/D69417





More information about the llvm-commits mailing list