[PATCH] D37957: [TableGen] Some simple optimizations to TableGen execution time

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 20 11:30:58 PDT 2017


zturner marked an inline comment as done.
zturner added inline comments.


================
Comment at: llvm/include/llvm/ADT/DenseSet.h:192
 
+  bool operator==(const DenseSetImpl<ValueT, MapTy, ValueInfoT> &Other) const {
+    if (size() != Other.size())
----------------
rnk wrote:
> What made us need this?
It was no longer needed after Krzysztof's other changes.  I removed this in the version that I committted.


Repository:
  rL LLVM

https://reviews.llvm.org/D37957





More information about the llvm-commits mailing list